[clang] [clang][ASTMatcher] Fix execution order of hasOperands submatchers (PR #104148)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 15 06:37:59 PDT 2024
https://github.com/AaronBallman commented:
Oh, that's a good catch -- thank you for the fix!
Can you add a new test case in https://github.com/llvm/llvm-project/blob/main/clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp so we don't regress this behavior accidentally in the future?
Also, please add a release note in https://github.com/llvm/llvm-project/blob/026d963cb004689477d2b5798cbba5ad41c25a70/clang/docs/ReleaseNotes.rst?plain=1#L347 so users know about the fix?
https://github.com/llvm/llvm-project/pull/104148
More information about the cfe-commits
mailing list