[llvm] ca70cee - Revert "[RISCV] Relax opcode restriction for .insn."

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 10:57:54 PDT 2023


Author: Craig Topper
Date: 2023-03-27T10:57:45-07:00
New Revision: ca70ceeb0f2c2362b37512a81a17d59e17b5fb26

URL: https://github.com/llvm/llvm-project/commit/ca70ceeb0f2c2362b37512a81a17d59e17b5fb26
DIFF: https://github.com/llvm/llvm-project/commit/ca70ceeb0f2c2362b37512a81a17d59e17b5fb26.diff

LOG: Revert "[RISCV] Relax opcode restriction for .insn."

This reverts commit 5c5fe3afff12b41d34477977c07c71337fa4ba9d.

This is failing on the build bots

Added: 
    

Modified: 
    llvm/test/MC/RISCV/insn-invalid.s

Removed: 
    


################################################################################
diff  --git a/llvm/test/MC/RISCV/insn-invalid.s b/llvm/test/MC/RISCV/insn-invalid.s
index e9fdc4042c1d..82fdd7b579ea 100644
--- a/llvm/test/MC/RISCV/insn-invalid.s
+++ b/llvm/test/MC/RISCV/insn-invalid.s
@@ -14,12 +14,12 @@
 .insn q  0x13,  0,  a0, a1, 13, 14 # CHECK: :[[@LINE]]:7: error: invalid instruction format
 
 # Invalid immediate
-.insn i  0x99,  0, a0, 4(a1) # CHECK: :[[@LINE]]:10: error: opcode must be a valid opcode name or an immediate in the range [0, 127]
+.insn i  0x99,  0, a0, 4(a1) # CHECK: :[[@LINE]]:10: error: opcode must be in the range [0, 127]
 .insn r  0x33,  8,  0, a0, a1, a2 # CHECK: :[[@LINE]]:17: error: immediate must be an integer in the range [0, 7]
 .insn r4 0x43,  0,  4, fa0, fa1, fa2, fa3 # CHECK: :[[@LINE]]:21: error: immediate must be an integer in the range [0, 3]
 
 # Unrecognized opcode name
-.insn r UNKNOWN, 0, a1, a2, a3 #CHECK: :[[@LINE]]:9: error: opcode must be a valid opcode name or an immediate in the range [0, 127]
+.insn r UNKNOWN, 0, a1, a2, a3 #CHECK: :[[@LINE]]:9: error: opcode must be in the range [0, 127] and the lower 2 bits must be 0x3
 
 # Make fake mnemonics we use to match these in the tablegened asm match table isn't exposed.
 .insn_i  0x13,  0,  a0, a1, 13, 14 # CHECK: :[[@LINE]]:1: error: unknown directive


        


More information about the llvm-commits mailing list