[PATCH] Object: Move auxiliary symbol definitions from llvm-readobj

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Tue Mar 18 16:08:35 PDT 2014


  Please also git-clang-format the patch.


================
Comment at: include/llvm/Object/COFF.h:249
@@ -232,2 +248,3 @@
   support::ulittle32_t TagIndex;
   support::ulittle32_t Characteristics;
+  char                 unused[10];
----------------
Unused would be the correct name according to the LLVM style.

================
Comment at: include/llvm/Object/COFF.h:234
@@ +233,3 @@
+  support::ulittle32_t TotalSize;
+  support::ulittle32_t PointerToLinenumber;
+  support::ulittle32_t PointerToNextFunction;
----------------
Why rename from PointerToLineNumber?


http://llvm-reviews.chandlerc.com/D3105

ARCANIST PROJECT
  llvm



More information about the llvm-commits mailing list