CarlosAlbertoEnciso 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.

Added 2 test cases to cover when a class is define inside a function:
Using `CBase` and `CDerived` from the previous test cases:
- `CBase` defined at global scope and `CDerived` defined at function scope.
- `CBase` and `CDerived` both defined at function scope.

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