[PATCH] D59673: [Driver] Allow setting the DWO name DWARF attribute separately

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 21 15:28:36 PDT 2019


dblaikie added a comment.

Pleasue include mention of the bug (PR40276) in the commit message & clarify that while this is useful for some remote compilation models, it's not strictly necessary/the only way to do it (a remote compilation model that keeps relative paths and uses compilation-dir instead can work without this attribute)

Use llvm-dwarfdump rather than llvm-objdump to dump the contents of the debug_info section and test the dwo_name there (rather than dumping hex) & probably the objdump part of the test isn't needed? (& I guess there's an LLVM patch to add the rest of this functionality?)


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59673/new/

https://reviews.llvm.org/D59673





More information about the cfe-commits mailing list