[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-02-03 Thread Stephen Kelly via Phabricator via cfe-commits
steveire updated this revision to Diff 321233. steveire added a comment. Update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94131/new/ https://reviews.llvm.org/D94131 Files: clang-tools-extra/clang-tidy/bugprone/InfiniteLoopCheck.cpp

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-16 Thread Nathan James via Phabricator via cfe-commits
njames93 requested changes to this revision. njames93 added a comment. This revision now requires changes to proceed. Can you either update the description of this patch to include the binaryOperation changes, or remove those changes from here. Comment at:

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-16 Thread Stephen Kelly via Phabricator via cfe-commits
steveire updated this revision to Diff 317177. steveire added a comment. Update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94131/new/ https://reviews.llvm.org/D94131 Files: clang-tools-extra/clang-tidy/bugprone/InfiniteLoopCheck.cpp

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-11 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. In D94131#2481804 , @njames93 wrote: > Theres a few compile errors here in the pre merge. Is this patch based > against trunk or some

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-06 Thread Nathan James via Phabricator via cfe-commits
njames93 added a comment. Theres a few compile errors here in the pre merge. Is this patch based against trunk or some local branch? Comment at: clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp:313 // may be instantiated to use std::move() on

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-05 Thread Stephen Kelly via Phabricator via cfe-commits
steveire updated this revision to Diff 314739. steveire added a comment. Update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94131/new/ https://reviews.llvm.org/D94131 Files: clang-tools-extra/clang-tidy/bugprone/InfiniteLoopCheck.cpp

[PATCH] D94131: [clang-tidy] Use new mapAnyOf matcher

2021-01-05 Thread Stephen Kelly via Phabricator via cfe-commits
steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added subscribers: kbarton, xazax.hun, nemanjai. steveire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo