[llvm] r204621 - [mips] Add error message when trying to use $at in '.set noat' mode.

[Daniel Sanders]

Author: dsanders
Date: Mon Mar 24 11:48:01 2014
New Revision: 204621

URL: http://llvm.org/viewvc/llvm-project?rev=204621&view=rev
Log:
[mips] Add error message when trying to use $at in '.set noat' mode.

Summary:
Patch by David Chisnall
His work was sponsored by: DARPA, AFRL

Differential Revision: http://llvm-reviews.chandlerc.com/D3158

Added:
    llvm/trunk/test/MC/Mips/mips-noat.s
Modified:
    llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp

Modified: llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp?rev=204621&r1=204620&r2=204621&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp (original)
+++ llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp Mon Mar 24 11:48:01 2014
@@ -1174,7 +1174,12 @@ bool MipsAssemblerOptions::setATReg(unsi
   return true;
 }
 
-int MipsAsmParser::getATReg() { return Options.getATRegNum(); }
+int MipsAsmParser::getATReg() {
+  int AT = Options.getATRegNum();
+  if (AT == 0)
+    TokError("Pseudo instruction requires $at, which is not available");
+  return AT;
+}
 
 unsigned MipsAsmParser::getReg(int RC, int RegNo) {
   return *(getContext().getRegisterInfo()->getRegClass(RC).begin() + RegNo);

Added: llvm/trunk/test/MC/Mips/mips-noat.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/Mips/mips-noat.s?rev=204621&view=auto
==============================================================================
--- llvm/trunk/test/MC/Mips/mips-noat.s (added)
+++ llvm/trunk/test/MC/Mips/mips-noat.s Mon Mar 24 11:48:01 2014
@@ -0,0 +1,27 @@
+# RUN: not llvm-mc %s -triple=mips-unknown-linux 2>%t0 | FileCheck %s
+# RUN: FileCheck -check-prefix=ERROR %s < %t0
+
+# We start with $at enabled
+# CHECK-LABEL: test1:
+# CHECK:  lui   $1, 1
+# CHECK:  addu  $1, $1, $2
+# CHECK:  lw    $2, 0($1)
+test1:
+        lw      $2, 65536($2)
+
+# Check that using $at when .set noat is in effect is an error.
+# FIXME: It would be better if the error pointed at the mnemonic instead of the newline
+# ERROR: mips-noat.s:[[@LINE+5]]:1: error: Pseudo instruction requires $at, which is not available
+# ERROR-NOT: error
+test2:
+        .set noat
+        lw      $2, 65536($2)
+
+# Can we switch it back on successfully?
+# CHECK-LABEL: test3:
+# CHECK:  lui   $1, 1
+# CHECK:  addu  $1, $1, $2
+# CHECK:  lw    $2, 0($1)
+test3:
+        .set at
+        lw      $2, 65536($2)