Maoyao233 wrote:

> > For reference, I was able to construct a bug in ASTMatcher with the same 
> > root cause: because RAV misses lambda bodies, ASTMatcher's 
> > `hasDescendant`/`forEachDescendant` cannot reach them, so it silently 
> > produces unexpected results. Of course, it could use the same approach as 
> > this PR as a workaround. I’m not sure about its severity or impact. Should 
> > I open a separate follow-up issue to discuss it?
> 
> Did you consider fixing how lambdas get merged across modules, e.g. by 
> assigning them the same body?

Make sence, but I have to consider it carefully. I will close this PR 
temporarily. Thanks.

https://github.com/llvm/llvm-project/pull/237067
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to