[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