[PATCH] D115325: [DWARF] Fix PR51087 Extraneous enum record in DWARF with type units
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 10 12:57:20 PST 2022
dblaikie added a comment.
In D115325#3232487 <https://reviews.llvm.org/D115325#3232487>, @dblaikie wrote:
> Hmm... maybe we disable pubnames when there are type units enabled? I hadn't remembered that, and not sure what that does for Split DWARF + type units... because I could've sworn pubnames were /necessary/ for Split DWARf, at least with GDB... so many layers.
>
> No accelerator tables if type units are enabled, as DWARF v4 type units are not compatible with accelerator tables.
>
> (according to `llvm/test/DebugInfo/X86/accel-tables.ll)
>
> *more testing*
OK, so looks like only -ggnu-pubname / .debug_gnu_pubnames / .debug_gnu_pubtypes is supported with Type Units enabled. Though in theory/according to that comment ^ it should be OK to support debug_names with type units.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115325/new/
https://reviews.llvm.org/D115325
More information about the llvm-commits
mailing list