[clang] Fix hasName matcher assertion with inline namespaces (PR #100975)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 29 05:36:35 PDT 2024


AaronBallman wrote:

> > I can't get the assertion to reproduce, but I can see the behavioral change with the first matcher: https://godbolt.org/z/TcsYPoGEh
> > So this should probably come with a release note for the fix?
> 
> The assertions are only shown on debug build, but it comes from [here](https://github.com/llvm/llvm-project/blob/9e3f5ae48fa75d5db8b132162d96004191618956/clang/lib/ASTMatchers/ASTMatchersInternal.cpp#L678)

That should still trigger with an assertions build though, right?

> Which section in the release notes do changes to AST matchers belong in

Somewhere around here: https://github.com/llvm/llvm-project/blob/77655f42d58e85875c4b4e28a73208b64a653c2a/clang/docs/ReleaseNotes.rst?plain=1#L247

https://github.com/llvm/llvm-project/pull/100975


More information about the cfe-commits mailing list