[PATCH] D15575: [ELF] Allow target to configure ELF header flags in the output file

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 16 09:30:02 PST 2015


ruiu added inline comments.

================
Comment at: ELF/Target.cpp:1257-1261
@@ -1256,2 +1256,7 @@
   GotHeaderEntriesNum = 2;
+  // FIXME: In fact ELF flags depends on ELF flags of input object files
+  // and selected emulation.
+  ELFFlags = EF_MIPS_ABI_O32 | EF_MIPS_CPIC | EF_MIPS_ARCH_32R2;
+  if (Config->Shared)
+    ELFFlags |= EF_MIPS_PIC;
 }
----------------
Instead of assigning a value to ELFFlags member, can you move this code to getELFFlags and return a new value without assignment?


Repository:
  rL LLVM

http://reviews.llvm.org/D15575





More information about the llvm-commits mailing list