[clang-tools-extra] [clang-tidy] In C++17, callee is guaranteed to be sequenced before arguments. (PR #93623)

Kefu Chai via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 8 17:08:21 PDT 2024


================
@@ -175,6 +218,10 @@ bool UseAfterMoveFinder::findInternal(const CFGBlock *Block,
             MovingCall != nullptr &&
             Sequence->potentiallyAfter(MovingCall, Use);
 
+        // We default to false here and change this to true if required in
+        // find().
+        TheUseAfterMove->UseHappensInLaterLoopIteration = false;
+
----------------
tchaikov wrote:

> for better readability. but the `optional<>` refactor is still a nice-to-have, IMHO.

created #98100

https://github.com/llvm/llvm-project/pull/93623


More information about the cfe-commits mailing list