diff options
| author | Thom Chiovoloni <chiovolonit@gmail.com> | 2020-12-02 15:16:12 -0800 |
|---|---|---|
| committer | Joshua Nelson <jyn514@gmail.com> | 2021-08-16 03:55:27 +0000 |
| commit | 295225b660e1ddfd4a476a14aa12b37ae140f91d (patch) | |
| tree | 115d01a5d050c4dd85dc56826e9cf71bd5af9dbc /src/test/codegen/src-hash-algorithm | |
| parent | c9e45e47d54b20d6c442c6ee1bc2c192e3181fdc (diff) | |
| download | rust-295225b660e1ddfd4a476a14aa12b37ae140f91d.tar.gz rust-295225b660e1ddfd4a476a14aa12b37ae140f91d.zip | |
Uplift the `invalid_atomic_ordering` lint from clippy to rustc
- Deprecate clippy::invalid_atomic_ordering - Use rustc_diagnostic_item for the orderings in the invalid_atomic_ordering lint - Reduce code duplication - Give up on making enum variants diagnostic items and just look for `Ordering` instead I ran into tons of trouble with this because apparently the change to store HIR attrs in a side table also gave the DefIds of the constructor instead of the variant itself. So I had to change `matches_ordering` to also check the grandparent of the defid as well. - Rename `atomic_ordering_x` symbols to just the name of the variant - Fix typos in checks - there were a few places that said "may not be Release" in the diagnostic but actually checked for SeqCst in the lint. - Make constant items const - Use fewer diagnostic items - Only look at arguments after making sure the method matches This prevents an ICE when there aren't enough arguments. - Ignore trait methods - Only check Ctors instead of going through `qpath_res` The functions take values, so this couldn't ever be anything else. - Add if_chain to allowed dependencies - Fix grammar - Remove unnecessary allow
Diffstat (limited to 'src/test/codegen/src-hash-algorithm')
0 files changed, 0 insertions, 0 deletions
