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

Tom Tromey via llvm-commits llvm-commits at lists.llvm.org
Tue May 13 09:09:46 PDT 2025


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


More information about the llvm-commits mailing list