[PATCH] First step towards PR21189 -- Teach llvm-readobj to dump bits of COFF symbol subsections required to debug using VS2012+

Nico Rieck nico.rieck at gmail.com
Fri Oct 17 10:43:53 PDT 2014


================
Comment at: tools/llvm-readobj/COFFDumper.cpp:621
@@ +620,3 @@
+    case COFF::DEBUG_SYMBOL_TYPE_PROC_START: {
+      ListScope S(W, "ProcStart");
+      if (InFunctionScope) {
----------------
Minor nit. This should be DictScope because the content is not a list.

================
Comment at: tools/llvm-readobj/COFFDumper.cpp:648
@@ +647,3 @@
+    case COFF::DEBUG_SYMBOL_TYPE_PROC_END: {
+      ListScope S(W, "ProcEnd");
+      if (!InFunctionScope) {
----------------
Ditto here. Does this subsection have any content not added yet? If not, I'd just print "ProcEnd".

http://reviews.llvm.org/D5755






More information about the llvm-commits mailing list