[PATCH] D21779: [LTO] Infer EKind/EMachine from Bitcode files

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 06:33:18 PDT 2016


dsanders added a subscriber: dsanders.

================
Comment at: lld/trunk/ELF/InputFiles.cpp:559-562
@@ +558,6 @@
+  Triple TheTriple(TripleStr);
+  bool Is64Bits = TheTriple.isArch64Bit();
+  if (TheTriple.isLittleEndian())
+    return Is64Bits ? ELF64LEKind : ELF32LEKind;
+  return Is64Bits ? ELF64BEKind : ELF32BEKind;
+}
----------------
Sorry for the late comment but Triple::isArch64Bit() doesn't correspond to ELF64 or ELF32 on MIPS. In particular, the triple used for the N32 ABI will return true for isArch64Bit() but this ABI uses ELF32 objects. If lld doesn't support the N32 ABI yet then the current code is ok for the remaining two ABI's (O32 and N64).


Repository:
  rL LLVM

http://reviews.llvm.org/D21779





More information about the llvm-commits mailing list