[PATCH] D15740: [mips] Add SHF_MIPS_GPREL flag to the MIPS .sbss and .sdata sections

Simon Atanasyan via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 11:37:16 PST 2016


atanasyan added inline comments.

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:803
@@ -802,2 +802,3 @@
   LLVM_READOBJ_ENUM_ENT(ELF, SHF_MIPS_NOSTRIP    ),
+  LLVM_READOBJ_ENUM_ENT(ELF, SHF_MIPS_GPREL      ),
   LLVM_READOBJ_ENUM_ENT(ELF, SHF_AMDGPU_HSA_GLOBAL),
----------------
khemant wrote:
> Probably I am not understanding this right,  this flag is in SHF_MASKPROC bit fields. Shouldn't the flag printer (for instance in section printer)  be checking processor printing the section flags ? Adding it here makes any processor that may use the same field not be able to use the enum string other than SHF_MIPS_GPREL.
Good point, thanks. I will fix it.


Repository:
  rL LLVM

http://reviews.llvm.org/D15740





More information about the llvm-commits mailing list