[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