[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