aaron.ballman added inline comments.
================ Comment at: clang/lib/ASTMatchers/ASTMatchFinder.cpp:245 + ScopedIncrement ScopedDepth(&CurrentDepth); + if (!match(*Node->getLoopVariable()) || !match(*Node->getRangeInit()) || + !match(*Node->getBody())) ---------------- Should we be traversing the init statement before the loop variable so that the traversals happen in lexical order? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94031/new/ https://reviews.llvm.org/D94031 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits