[PATCH] D18224: llvm-readobj: GNU style symbols printing

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 08:09:34 PDT 2016


rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM with nits.


================
Comment at: tools/llvm-readobj/ELFDumper.cpp:2549
@@ +2548,3 @@
+        Symbol, FirstSym, this->dumper()->getShndxTable());
+  default:
+    if (SectionIndex >= ELF::SHN_LOPROC && SectionIndex <= ELF::SHN_HIPROC)
----------------
Add a fallthrough comment.

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:2588
@@ +2587,3 @@
+  }
+  Field fields[8] = {0,         8,         17 + Bias, 23 + Bias,
+                     31 + Bias, 38 + Bias, 47 + Bias, 51 + Bias};
----------------
Start variable name with uppper case.


Repository:
  rL LLVM

http://reviews.llvm.org/D18224





More information about the llvm-commits mailing list