diff options
| author | Dylan DPC <dylan.dpc@gmail.com> | 2021-02-14 16:54:42 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-14 16:54:42 +0100 |
| commit | ac1d26bcd3a26523c7e7ebb2dd0b37d6b2834291 (patch) | |
| tree | 96024d4a5568ea17d21e71efd130e71371005e31 /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp | |
| parent | 29ed864dc3a066487180d08dcf8b64c6afedebba (diff) | |
| parent | 9f0e1d4921bbb40fea71594d7c599c09ab513232 (diff) | |
| download | rust-ac1d26bcd3a26523c7e7ebb2dd0b37d6b2834291.tar.gz rust-ac1d26bcd3a26523c7e7ebb2dd0b37d6b2834291.zip | |
Rollup merge of #80920 - rylev:check_attr-refactor, r=davidtwco
Visit more targets when validating attributes This begins to address #80048, allowing for additional validation of attributes. There are more refactorings that can be done, though I think they should be tackled in additional PRs: * ICE when a builtin attribute is encountered that is not checked * Move some of the attr checking done `ast_validation` into `rustc_passes` * note that this requires a bit of additional refactoring, especially of extern items which currently parse attributes (and thus are a part of the AST) but do not possess attributes in their HIR representation. * Rename `Target` to `AttributeTarget` * Refactor attribute validation completely to go through `Visitor::visit_attribute`. * This would require at a minimum passing `Target` into this method which might be too big of a refactoring to be worth it. * It's also likely not possible to do all the validation this way as some validation requires knowing what other attributes a target has. r? `@davidtwco`
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
