[mips] '.set at=$0' should be equivalent to '.set noat'
authorDaniel Sanders <daniel.sanders@imgtec.com>
Tue, 25 Mar 2014 13:01:06 +0000 (13:01 +0000)
committerDaniel Sanders <daniel.sanders@imgtec.com>
Tue, 25 Mar 2014 13:01:06 +0000 (13:01 +0000)
Differential Revision: http://llvm-reviews.chandlerc.com/D3171

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@204714 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/Mips/AsmParser/MipsAsmParser.cpp
test/MC/Mips/mips-noat.s
test/MC/Mips/set-at-directive-explicit-at.s

index 74e19c5be33d57f74cdd5450ca6b3a59b7790e7c..078b554e2e07cc6ff2e0fb0123738a5d39fd5399 100644 (file)
@@ -2352,7 +2352,7 @@ bool MipsAsmParser::parseSetAtDirective() {
       return false;
     }
 
-    if (AtRegNo < 1 || AtRegNo > 31) {
+    if (AtRegNo < 0 || AtRegNo > 31) {
       reportParseError("unexpected token in statement");
       return false;
     }
index 152a1ee4067f09dc62473e865efac5867559e58f..b83c517807769fa129bd6c45708ce847b6b47c9f 100644 (file)
@@ -1,7 +1,8 @@
 # RUN: not llvm-mc %s -triple=mips-unknown-linux 2>%t0 | FileCheck %s
 # RUN: FileCheck -check-prefix=ERROR %s < %t0
+# Check that using the assembler temporary when .set noat is in effect is an error.
 
-# We start with $at enabled
+# We start with the assembler temporary enabled
 # CHECK-LABEL: test1:
 # CHECK:  lui   $1, 1
 # CHECK:  addu  $1, $1, $2
@@ -9,10 +10,8 @@
 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
+# ERROR: mips-noat.s:[[@LINE+4]]:1: error: Pseudo instruction requires $at, which is not available
 test2:
         .set noat
         lw      $2, 65536($2)
@@ -25,3 +24,11 @@ test2:
 test3:
         .set at
         lw      $2, 65536($2)
+
+# FIXME: It would be better if the error pointed at the mnemonic instead of the newline
+# ERROR: mips-noat.s:[[@LINE+4]]:1: error: Pseudo instruction requires $at, which is not available
+test4:
+        .set at=$0
+        lw      $2, 65536($2)
+
+# ERROR-NOT: error
index b8c8adc8818c98b6db82279f157a0e99c1a3a47d..71f1a98fae49e2cfb3106caba01a39500eeda0fa 100644 (file)
@@ -26,6 +26,9 @@ foo:
 # CHECK:   jr    $1                      # encoding: [0x08,0x00,0x20,0x00]
     .set noat
     jr    $at
+# CHECK:   jr    $1                      # encoding: [0x08,0x00,0x20,0x00]
+    .set at=$0
+    jr    $at
 
 # CHECK:   jr    $16                     # encoding: [0x08,0x00,0x00,0x02]
 # WARNINGS: :[[@LINE+2]]:12: warning: Used $16 with ".set at=$16"