[PATCH] [Mips][ABI]The ELF container needs to depend on the ABI rather than the target triple.
Eric Christopher
echristo at gmail.com
Thu Feb 19 11:04:47 PST 2015
Let's split this patch into two pieces to make the review easier - part of this patch is ok (#1) and I'd like to chat about #2 :)
1. Passing MCOptions to the backend,
2. Using it in the mips backend
Thanks!
-eric
================
Comment at: lib/Target/Mips/MCTargetDesc/MipsAsmBackend.h:33
@@ -32,3 +32,3 @@
bool Is64Bit; // 32 or 64 bit words
-
+ bool IsN64; // Is ABI n64
public:
----------------
You could probably store just the ABI here and unify the previous 3 class variables with just the triple rather than OSType.
http://reviews.llvm.org/D6476
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list