[PATCH] D97243: [clang-tidy] Remove IncludeInserter from MoveConstructorInit check.

2021-02-23 Thread Nathan James via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe96f9cca3b1b: [clang-tidy] Remove IncludeInserter from MoveConstructorInit check. (authored by njames93). Changed prior to commit: https://reviews.llvm.org/D97243?vs=325626&id=325757#toc Repository:

[PATCH] D97243: [clang-tidy] Remove IncludeInserter from MoveConstructorInit check.

2021-02-23 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thanks for the cleanup! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97243/new/ https://reviews.llvm.org/D97243 __

[PATCH] D97243: [clang-tidy] Remove IncludeInserter from MoveConstructorInit check.

2021-02-22 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 325626. njames93 added a comment. Remove unnessary headers. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97243/new/ https://reviews.llvm.org/D97243 Files: clang-tools-extra/clang-tidy/performance/MoveConst

[PATCH] D97243: [clang-tidy] Remove IncludeInserter from MoveConstructorInit check.

2021-02-22 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added reviewers: alexfh, aaron.ballman. Herald added a subscriber: xazax.hun. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This check registers an IncludeInserter, however the check itsel