[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