move testfiles to "lintcheck" and fix more paths

gitignore: add lintchecks target dir

lintcheck: make sure we lauch from the repo root

This will terminate the program if run via "cargo run".
"cargo run" does currently not work because at least a bunch of paths do not take that into account.

lintcheck: move out of clippy-dev into own crate

Auto merge of #6881 - flip1995:rustup, r=flip1995

Rustup

changelog: none

Fix remaining dogfood errors (internal lints)

Bump nightly version => 2021-03-11

Merge remote-tracking branch 'upstream/master' into rustup

Auto merge of #6814 - hyd-dev:issue-6486, r=flip1995

Fix false positives on procedural macros of `missing_inline_in_public_items` lint

Fixes #6486.

changelog: Fix false positives on procedural macros of `missing_inline_in_public_items` lint.

Auto merge of #6794 - camsteffen:needless-borrowed-ref, r=flip1995

Improve needless_borrowed_ref docs

changelog: none

I think "borrowed ref" is a confusing description for this lint. Destructuring a reference is the opposite of borrowing. So I updated the wording throughout the docs. Unfortunately this nit applies to the name of the lint itself, but I won't bother changing that. One motivation for these changes is to clarify the difference between this lint and `needless_borrow` (they are actually quite different). Let me know if I need to clarify anything or if you disagree with any changes.

Auto merge of #79519 - cjgillot:noattr, r=wesleywiser

Store HIR attributes in a side table

Same idea as #72015 but for attributes.
The objective is to reduce incr-comp invalidations due to modified attributes.
Notably, those due to modified doc comments.

Implementation:
- collect attributes during AST->HIR lowering, in `LocalDefId -> ItemLocalId -> &[Attributes]` nested tables;
- access the attributes through a `hir_owner_attrs` query;
- local refactorings to use this access;
- remove `attrs` from HIR data structures one-by-one.

Change in behaviour:
- the HIR visitor traverses all attributes at once instead of parent-by-parent;
- attribute arrays are sometimes duplicated: for statements and variant constructors;
- as a consequence, attributes are marked as used after unused-attribute lint emission to avoid duplicate lints.

~~Current bug: the lint level is not correctly applied in `std::backtrace_rs`, triggering an unused attribute warning on `#![no_std]`. I welcome suggestions.~~

Auto merge of #6878 - flip1995:authors, r=Manishearth

Set Clippy authors to "The Rust Clippy Developers"

Clippy has grown enough, that putting specific people in the "authors"
field isn't warranted anymore.

As a heads-up: `@Manishearth` `@llogiq` `@birkenfeld` `@mcarton` `@oli-obk` `@phansch` `@matthiaskrgr` your names will be removed from one or more of the `Cargo.toml` files of Clippy. This of course does not mean that we value your previous work on Clippy any less :heart:

As per our discussion in today's meeting: https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Meeting.202021-03-09/near/229502514

changelog: none

Remove hir::Expr::attrs.

Remove hir::Item::attrs.

Remove hir::ImplItem::attrs.

Remove hir::TraitItem::attrs.

Remove hir::StructField::attrs.

Remove hir::Variant::attrs.

Remove hir::Arm::attrs.

Remove hir::Crate::attrs.

Remove hir::Local::attrs.

Remove hir::StmtKind::attrs.

Do not store attrs in FnKind.

Simplify clippy author.

Switch to changing cp_non_overlap in tform

It was suggested to lower this in MIR instead of ssa, so do that instead.

Update cranelift

Update match branches

This updates all places where match branches check on StatementKind or UseContext.
This doesn't properly implement them, but adds TODOs where they are, and also adds some best
guesses to what they should be in some cases.

Set Clippy authors to "The Rust Clippy Developers"

Clippy has grown enough, that putting specific people in the "authors"
field is warranted anymore.

Auto merge of #6876 - giraffate:improve_doc_on_map_flatten, r=flip1995

Improve doc on `map_flatten`

Fix https://github.com/rust-lang/rust-clippy/issues/6870. It's because this doc lacks the description that this lint is also used for `Option`.

changelog: none

Improve doc on `map_flatten`

Auto merge of #6866 - anall:ice6840, r=flip1995

Fix ICE 6840 - make is_normalizable more strict

fixes #6840

make `is_normalizable` more strict, which should catch this ICE and related cases

changelog: Fix ICE in [`zero_sized_map_values`]

add comment for when  can be removed

Auto merge of #6868 - Jarcho:lang_item, r=flip1995

Don't assume lang items will exist.

~~Should fix lintcheck warnings caused by #6823~~
See below

changelog: None

Auto merge of #6873 - Y-Nak:refactor-casts-lint, r=flip1995

Refactor casts lint

Ref: #6724

Changes:
1. Separate the `casts` group from the `types` group.
2. Reorganize the lints of the `casts` group into their own modules.

Notes:
1. I didn't `fix` #6874 in order to maintain this PR as small as possible.

---
changelog: none

Move ptr_as_ptr to its own module

Rollup merge of #82048 - mark-i-m:or-pat-type-ascription, r=petrochenkov

or-patterns: disallow in `let` bindings

~~Blocked on https://github.com/rust-lang/rust/pull/81869~~

Disallows top-level or-patterns before type ascription. We want to reserve this syntactic space for possible future generalized type ascription.

r? ``@petrochenkov``

Move char_lit_as_u8 to its own module

Move cast_ref_to_mut to its own module

Move cast_ptr_alignment to its own module

Move fn_to_numeric_cast_with_truncation to its own module

Move fn_to_numeric_cast to its own module

Remove 'lint_numeric_casts'

Move unnecessary_cast to its own module

Move cast_possible_wrap to its own module

Move cast_sign_loss to its own module

Move cast_possible_truncation to its own module

Auto merge of #6869 - DJMcNab:rust-analyzer-private, r=phansch

Opt-in to rustc_private for `rust-analyzer`

rust-analyzer/rust-analyzer#7891

changelog: none

This will also help priroda and any other package which depends on the `miri` library crate.

use `.all` instead of negative use of `.any`

use TyS::walk

Move cast_lossless to its own module

Move cast_precision_loss to its own module

Move 'is_isize_or_usize' to clippy_utils

Separate lints of casts group from types group

Auto merge of #6871 - camsteffen:redundant-closure-macro, r=Manishearth

Fix redundant closure with macros

changelog: Fix redundant_closure FPs with macros

Fixes #6732
Fixes #6850
Fixes #4354 (addresses the error message confusion)

Improve the redundant_closure message

clippy: fix or-pattern in let binding

Fix redundant closure with macros

Opt-in to rustc_private for `rust-analyzer`

rust-analyzer/rust-analyzer#7891

Don't assume lang items will exist.

Auto merge of #6725 - Y-Nak:refactor-types-lints, r=flip1995

Refactor types lints

Ref #6724.
As described in #6724, `types.rs` contains many groups inside it.
In this PR, I reorganize the lints of the `types` group into their own modules.

changelog: none

Fix some lints in types that fail dogfood

Import declared lints at the top of the file

Remove unused is_local from borrowed_box

Move borrowed_box to its own module

Move linked_list to its own module

Move option_option to its own module

Add flags to detect lints are triggered

Move vec_box to its own module

Move rc_buffer to its own module

Move redundant_allocation to its own module

Move box_vec to its own module

Create types dir and move old module under it

Auto merge of #6834 - hyd-dev:clippy-args, r=phansch,flip1995,oli-obk

Let Cargo track CLIPPY_ARGS

This PR makes `clippy-driver` emit `CLIPPY_ARGS` in its `dep-info` output.

Just like #6441, this allows this workflow to work:
```shell
cargo clippy # warning: empty `loop {}` wastes CPU cycles
cargo clippy -- -A clippy::empty_loop # no warnings emitted
```
But without rebuilding all dependencies.

cc https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/CLIPPY_ARGS.20is.20not.20tracked.20by.20Cargo/near/228599088

changelog: Cargo now re-runs Clippy if arguments after `--` provided to `cargo clippy` are changed.

Don't panic if `CLIPPY_ARGS` is not Unicode

Move `test_no_deps_ignores_path_deps_in_workspaces()` out of `dogfood_subprojects()`

Rename `ClippyArgsCallbacks` to `RustcCallbacks`

Auto merge of #82727 - oli-obk:shrinkmem, r=pnkfelix

Test the effect of shrinking the size of Rvalue by 16 bytes

r? `@ghost`

Auto merge of #6864 - giraffate:fix_doc_adding_a_new_lint, r=Manishearth

Add msrv to contents in adding lints md

changelog: none

make is_normalizable more strict

Add msrv to contents in adding lints md

Auto merge of #6807 - anall:feature/use_new_diagnostics, r=Manishearth

migrate paths to newly-added diagnostic items

This gets rid of the following paths:
  * OS_STRING
  * TO_OWNED
  * TO_STRING

Removes some usages of:
 * PATH_BUF

Per #5393

also removes unneeded `is_ty_param_path` from `clippy_lints::types` and relocates `is_ty_param_lang_item` and `is_ty_param_diagnostic_item` to `clippy_utils`.

changelog: none

relocate functions from `clippy_lints::types`

relocate `is_ty_param_lang_item` and `is_ty_param_diagnostic_item` to `clippy_utils`

migrate paths to newly-added diagnostic items

This gets rid of the following paths:
  * OS_STRING
  * TO_OWNED
  * TO_STRING

Also removes some usages of:
  * PATH_BUF

And the now completely unused `clippy_lints::types::is_ty_param_path`

Auto merge of #6853 - Jarcho:len_without_is_empty_fp, r=Manishearth

`len_without_is_empty` will now consider multiple impl blocks

fixes #1562

This also reverts #1559 as the `#[allow]` now works on the `len` method. A note has also been added to point out where the `empty` method is, if it exists.

changelog: `len_without_is_empty` will now consider multiple impl blocks
changelog: `len_without_is_empty` will now consider `#[allow]` on both the `len` method, and the type definition

`len_without_is_empty` will now consider multiple impl blocks
`len_without_is_empty` will now consider `#[allow]` on both the `len` method, and the type definition

Auto merge of #6823 - Jarcho:diagnostic_items, r=phansch

Use diagnostic or language items instead of paths

I think that gets everything except ones used in a list of paths to check.

changelog: none

Use `LanguageItems::require`

Auto merge of #6854 - mgacek8:6844_fix_doc_for_useless_format_lint, r=phansch

useless_format: fix examples in the description

fixes #6844
changelog: useless_format: fix examples in the description

Auto merge of #6848 - matthiaskrgr:lintcheck_clippyfix, r=llogiq

lintcheck: add --fix mode which tries to apply lint suggestions to th…

…e sources and prints a warning if that fails

Great for spotting false positives/broken suggestions of applicable lints.

There are false positives though because I'm not sure yet how to silence rustc warnings while keeping clippy warnings.
Sometimes rustc makes a suggestion that fails to apply and the implementation does not differentiate between clippy and rustc warnings when applying lint suggestions.

changelog: none

Rollup merge of #82736 - spastorino:mir-opt-level-perf-changes, r=oli-obk

Bump optimization from mir_opt_level 2 to 3 and 3 to 4 and make "release" be level 2 by default

r? `@oli-obk`

Make clippy set mir_opt_level using Option

useless_format: fix examples in the description

"Good" example was something not acceptable by the useless_format lint.

Auto merge of #6852 - camsteffen:avoid-mir, r=Manishearth

Remove a couple MIR usages

changelog: none

We use MIR to get the return type of a closure/function in a couple places. But typeck seems like a better approach.

This is the easy part of #6080.

Also did a tiny cleanup with `typeck` -> `typeck_body`.

Use typeck_body

Avoid mir in missing_errors_doc

Avoid mir in implicit_return

lintcheck: add --fix mode which tries to apply lint suggestions to the sources and prints a warning if that fails

Great for spotting false positives/broken suggestions of applicable lints.

There are false positives though becasue I'm not sure yet how to silence rustc warnings while keeping clippy warnings.
Sometimes rustc makes a suggestion that fails to apply and the implementation does not differenciate between clippy and rustc warnings when applying lint suggestions.

changelog: none

Auto merge of #6849 - flip1995:dogfood-fix, r=matthiaskrgr

Dogfood and CI fixes

The CI fix is practically #6829 rebased and squashed into one commit

Dogfood fix is a follow up of #6802

r? `@matthiaskrgr` for lintcheck changes

(best reviewed with whitespace changes hidden)

changelog: none

Extract directory creation into its own function

Fix dogfood errors in clippy_dev