[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