[PATCH] D26283: [ELF] - Partial support of --gdb-index command line option (Part 3).

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 10:13:39 PST 2016


clayborg added a comment.

So it looks like it is down to wether we need to actually do the pubnames parsing stuff in lib/DebugInfo/DWARF or not. I'll let other make that call.



================
Comment at: ELF/GdbIndex.cpp:118
+      }
+    }
+  }
----------------
I'll let others comment on what is right to do as I am just starting out reviewing things in LLVM and don't know all the ropes. David?


https://reviews.llvm.org/D26283





More information about the llvm-commits mailing list