[clang] [llvm] [clang][DebugInfo] Add symbol for debugger with VTable information. (PR #130255)

Carlos Alberto Enciso via llvm-commits llvm-commits at lists.llvm.org
Thu May 15 07:00:48 PDT 2025


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


More information about the llvm-commits mailing list