[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