[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