tromey wrote:

Apologies if I missed it, but one thing I didn't see in the patch is a test for 
the case where a class is defined inside a function.

Given the discussion 
[here](https://discourse.llvm.org/t/rfc-dwarfdebug-fix-and-improve-handling-imported-entities-types-and-static-local-in-subprogram-and-lexical-block-scopes/68544),
 I guess this might not fully work correctly; but it seems to me that checking 
that the vtable symbol is global could be done and might provide some 
future-proofing.

Thanks.

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

Reply via email to