[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