[PATCH] [mips] [IAS] Don't do 64-bit expansions for LA of symbol.

Daniel Sanders daniel.sanders at imgtec.com
Thu Apr 30 05:49:52 PDT 2015


================
Comment at: lib/Target/Mips/AsmParser/MipsAsmParser.cpp:1930-1932
@@ +1929,5 @@
+    SMLoc IDLoc, SmallVectorImpl<MCInst> &Instructions) {
+  if (Is32BitSym && isGP64bit())
+    Warning(IDLoc,
+            "instruction loads a 32-bit address on a 64-bit architecture");
+
----------------
I don't think this warning is correct. O32 and N32 should be able to use 'la' on a 64-bit CPU. The warning makes sense for N64 though.

http://reviews.llvm.org/D9295

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list