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

khemant@codeaurora.org via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 11:24:18 PST 2016


khemant added a subscriber: khemant.

================
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),
----------------
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.


Repository:
  rL LLVM

http://reviews.llvm.org/D15740





More information about the llvm-commits mailing list