https://github.com/AaronBallman commented:

Thank you for working on a fix for this! Please be sure to update 
`clang/docs/ReleaseNotes.rst` so users know about the fix.

Test coverage for the changes is interesting in that I'm not certain there's a 
good test to add for this (the test would be one which is pretty expensive to 
run, I'd imagine). But if you can think of some good coverage to add to 
`clang/unittests/ASTMatchers/`, that'd be good to add as well.

https://github.com/llvm/llvm-project/pull/117143
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to