[llvm] [AsmPrinter][DebugNames] Implement DW_IDX_parent entries (PR #77457)

Greg Clayton via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 15:29:23 PST 2024


clayborg wrote:

> added test as suggested by @ayermolo I believe we're looking good to merge now?

The above forward declarations should _not_ end up in the name lookup as that will violate the DWARF spec. It is ok for an entry to be in the list of entries, but any foward declarations should not be able to be looked up by name. It _is_ ok for an entry to be in there for the parent index stuff, but we don't want to lookup MyStruct2 and find either entries from:
```
0x00000029:   DW_TAG_namespace
                DW_AT_name      ("MyNamespace")

0x0000002b:     DW_TAG_structure_type
                  DW_AT_declaration     (true)
                  DW_AT_signature       (0x8b108ad18e93cb95)

0x00000034:     DW_TAG_structure_type
                  DW_AT_declaration     (true)
                  DW_AT_signature       (0x690ca39e0eeaec8f)

0x0000003d:     NULL
```


https://github.com/llvm/llvm-project/pull/77457


More information about the llvm-commits mailing list