[clang] [ASTMatcher] Fix redundant macro expansion checks in getExpansionLocOfMacro (PR #117143)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 26 05:28:19 PST 2024


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


More information about the cfe-commits mailing list