jiixyj wrote:

> Can't we just avoid setting the ManglingContextDecl as a local VarDecl, 
> instead of filtering it out later? It seems that would be simpler.

Good idea! Do you think the logic checking for modules (i.e. the 
`ND->isInNamedModule() || ND->isFromGlobalModule()) && 
ND->isExternallyVisible())`) can then also be done at the place where 
`ManglingContextDecl` is set?

I'll try it and report back.

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

Reply via email to