yuxuanchen1997 wrote:

> > There's no second VarDecl
> 
> No.... This is not what merged mean. When we said merged, the redeclarations 
> should appear as a redecl link. It proves that exactly the merging is not 
> happening properly.

Actually, taking a step back. Why do we expect `S` to be considered redeclared? 
There's a single global decl in `a.h`. `b.h` imports `a.h`, and should `S` be 
even considered declared again?  It'd be surprising for the AST to be wrong on 
such a trivial case.

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

Reply via email to