[PATCH] D51420: [DebugInfo] Have the verifier accept missing linkage name for anything other than subprograms.

[Pavel Labath via Phabricator via llvm-commits]

labath added a comment.

You are correct that we were being too strict in our requirements here. Though if these structs do have linkage names they could actually be useful (eg as a substitute for the "qualified name hash" in lldb) and we could consider adding them as an extension.

For the test,have you tried writing some dwarf by hand (or more like, modifying an existing .s file to add a linkage name attribute. There should be some .s files in the debug names dumper test suite that can serve as a reasonable starting point)?


Repository:
  rL LLVM

https://reviews.llvm.org/D51420