[PATCH] D71185: [DWARF5] Start emitting DW_AT_dwo_name when -gdwarf-5 is specified.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 10:06:01 PST 2019


probinson added a comment.

Address the inline comments and LGTM.



================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:866
+                                       ? dwarf::DW_AT_dwo_name
+                                       : dwarf::DW_AT_GNU_dwo_name;
     NewCU.addUInt(Die, dwarf::DW_AT_GNU_dwo_id, dwarf::DW_FORM_data8,
----------------
This should be moved down under the 'if' so it's in the same scope as the only use.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:1110
+                                       ? dwarf::DW_AT_dwo_name
+                                       : dwarf::DW_AT_GNU_dwo_name;
     auto *SkCU = TheCU.getSkeleton();
----------------
This should be moved down under the 'if' so it's in the same scope as the only uses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71185





More information about the llvm-commits mailing list