njames93 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)

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

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

Reply via email to