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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to