From: Tim Northover Date: Tue, 6 May 2014 11:18:53 +0000 (+0000) Subject: AArch64/ARM64: produce more informative diagnostic assembling some immediates X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=6e64f90dc5c1869bef2ade6c596ad0eee50a9dc9;p=oota-llvm.git AArch64/ARM64: produce more informative diagnostic assembling some immediates No tests here, they'll be added when the entire neon-diagnostics.s test from AArch64 is enabled. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208079 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/ARM64/ARM64InstrFormats.td b/lib/Target/ARM64/ARM64InstrFormats.td index b67218cf5ea..26db1b2fac8 100644 --- a/lib/Target/ARM64/ARM64InstrFormats.td +++ b/lib/Target/ARM64/ARM64InstrFormats.td @@ -225,23 +225,16 @@ def imm0_65535 : Operand, ImmLeaf : AsmOperandClass { + let Name = "Imm" # Low # "_" # High; + let DiagnosticType = "InvalidImm" # Low # "_" # High; } +def Imm1_8Operand : AsmImmRange<1, 8>; +def Imm1_16Operand : AsmImmRange<1, 16>; +def Imm1_32Operand : AsmImmRange<1, 32>; +def Imm1_64Operand : AsmImmRange<1, 64>; + def MovZSymbolG3AsmOperand : AsmOperandClass { let Name = "MovZSymbolG3"; let RenderMethod = "addImmOperands"; @@ -386,10 +379,10 @@ def vecshiftR64Narrow : Operand, ImmLeaf; +def Imm0_15Operand : AsmImmRange<0, 15>; +def Imm0_31Operand : AsmImmRange<0, 31>; +def Imm0_63Operand : AsmImmRange<0, 63>; def vecshiftL8 : Operand, ImmLeaf opc, string asm, // FP Comparisons support only S and D element sizes. multiclass SIMDFPCmpTwoVector opc, string asm, SDNode OpNode> { + def v2i32rz : BaseSIMDCmpTwoVector<0, U, {S,0}, opc, V64, asm, ".2s", "0.0", v2i32, v2f32, OpNode>; diff --git a/lib/Target/ARM64/AsmParser/ARM64AsmParser.cpp b/lib/Target/ARM64/AsmParser/ARM64AsmParser.cpp index bad7a1ce58a..f2f666bc33f 100644 --- a/lib/Target/ARM64/AsmParser/ARM64AsmParser.cpp +++ b/lib/Target/ARM64/AsmParser/ARM64AsmParser.cpp @@ -3852,31 +3852,39 @@ bool ARM64AsmParser::showMatchError(SMLoc Loc, unsigned ErrCode) { case Match_InvalidSuffix: return Error(Loc, "invalid type suffix for instruction"); case Match_InvalidMemoryIndexedSImm9: - return Error(Loc, "index must be an integer in range [-256,255]."); + return Error(Loc, "index must be an integer in range [-256, 255]."); case Match_InvalidMemoryIndexed32SImm7: - return Error(Loc, "index must be a multiple of 4 in range [-256,252]."); + return Error(Loc, "index must be a multiple of 4 in range [-256, 252]."); case Match_InvalidMemoryIndexed64SImm7: - return Error(Loc, "index must be a multiple of 8 in range [-512,504]."); + return Error(Loc, "index must be a multiple of 8 in range [-512, 504]."); case Match_InvalidMemoryIndexed128SImm7: - return Error(Loc, "index must be a multiple of 16 in range [-1024,1008]."); + return Error(Loc, "index must be a multiple of 16 in range [-1024, 1008]."); case Match_InvalidMemoryIndexed8: - return Error(Loc, "index must be an integer in range [0,4095]."); + return Error(Loc, "index must be an integer in range [0, 4095]."); case Match_InvalidMemoryIndexed16: - return Error(Loc, "index must be a multiple of 2 in range [0,8190]."); + return Error(Loc, "index must be a multiple of 2 in range [0, 8190]."); case Match_InvalidMemoryIndexed32: - return Error(Loc, "index must be a multiple of 4 in range [0,16380]."); + return Error(Loc, "index must be a multiple of 4 in range [0, 16380]."); case Match_InvalidMemoryIndexed64: - return Error(Loc, "index must be a multiple of 8 in range [0,32760]."); + return Error(Loc, "index must be a multiple of 8 in range [0, 32760]."); case Match_InvalidMemoryIndexed128: - return Error(Loc, "index must be a multiple of 16 in range [0,65520]."); + return Error(Loc, "index must be a multiple of 16 in range [0, 65520]."); + case Match_InvalidImm0_7: + return Error(Loc, "immediate must be an integer in range [0, 7]."); + case Match_InvalidImm0_15: + return Error(Loc, "immediate must be an integer in range [0, 15]."); + case Match_InvalidImm0_31: + return Error(Loc, "immediate must be an integer in range [0, 31]."); + case Match_InvalidImm0_63: + return Error(Loc, "immediate must be an integer in range [0, 63]."); case Match_InvalidImm1_8: - return Error(Loc, "immediate must be an integer in range [1,8]."); + return Error(Loc, "immediate must be an integer in range [1, 8]."); case Match_InvalidImm1_16: - return Error(Loc, "immediate must be an integer in range [1,16]."); + return Error(Loc, "immediate must be an integer in range [1, 16]."); case Match_InvalidImm1_32: - return Error(Loc, "immediate must be an integer in range [1,32]."); + return Error(Loc, "immediate must be an integer in range [1, 32]."); case Match_InvalidImm1_64: - return Error(Loc, "immediate must be an integer in range [1,64]."); + return Error(Loc, "immediate must be an integer in range [1, 64]."); case Match_InvalidLabel: return Error(Loc, "expected label or encodable integer pc offset"); case Match_MRS: @@ -4416,6 +4424,10 @@ bool ARM64AsmParser::MatchAndEmitInstruction(SMLoc IDLoc, unsigned &Opcode, case Match_InvalidMemoryIndexed32SImm7: case Match_InvalidMemoryIndexed64SImm7: case Match_InvalidMemoryIndexed128SImm7: + case Match_InvalidImm0_7: + case Match_InvalidImm0_15: + case Match_InvalidImm0_31: + case Match_InvalidImm0_63: case Match_InvalidImm1_8: case Match_InvalidImm1_16: case Match_InvalidImm1_32: diff --git a/test/MC/ARM64/diags.s b/test/MC/ARM64/diags.s index 95b00a5b294..7d9ed5f2043 100644 --- a/test/MC/ARM64/diags.s +++ b/test/MC/ARM64/diags.s @@ -33,43 +33,43 @@ foo: ldur x0, [x1, #-257] -; CHECK-ERRORS: error: index must be a multiple of 8 in range [0,32760]. +; CHECK-ERRORS: error: index must be a multiple of 8 in range [0, 32760]. ; CHECK-ERRORS: ldr x0, [x0, #804] ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 4 in range [0,16380]. +; CHECK-ERRORS: error: index must be a multiple of 4 in range [0, 16380]. ; CHECK-ERRORS: ldr w0, [x0, #802] ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be an integer in range [-256,255]. +; CHECK-ERRORS: error: index must be an integer in range [-256, 255]. ; CHECK-ERRORS: ldr x0, [x0, #804]! ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be an integer in range [-256,255]. +; CHECK-ERRORS: error: index must be an integer in range [-256, 255]. ; CHECK-ERRORS: ldr w0, [w0, #301]! ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be an integer in range [-256,255]. +; CHECK-ERRORS: error: index must be an integer in range [-256, 255]. ; CHECK-ERRORS: ldr x0, [x0], #804 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be an integer in range [-256,255]. +; CHECK-ERRORS: error: index must be an integer in range [-256, 255]. ; CHECK-ERRORS: ldr w0, [w0], #301 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 4 in range [-256,252]. +; CHECK-ERRORS: error: index must be a multiple of 4 in range [-256, 252]. ; CHECK-ERRORS: ldp w3, w4, [x5, #11]! ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512,504]. +; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512, 504]. ; CHECK-ERRORS: ldp x3, x4, [x5, #12]! ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 16 in range [-1024,1008]. +; CHECK-ERRORS: error: index must be a multiple of 16 in range [-1024, 1008]. ; CHECK-ERRORS: ldp q3, q4, [x5, #12]! ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 4 in range [-256,252]. +; CHECK-ERRORS: error: index must be a multiple of 4 in range [-256, 252]. ; CHECK-ERRORS: ldp w3, w4, [x5], #11 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512,504]. +; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512, 504]. ; CHECK-ERRORS: ldp x3, x4, [x5], #12 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512,504]. +; CHECK-ERRORS: error: index must be a multiple of 8 in range [-512, 504]. ; CHECK-ERRORS: ldp q3, q4, [x5], #12 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: index must be an integer in range [-256,255]. +; CHECK-ERRORS: error: index must be an integer in range [-256, 255]. ; CHECK-ERRORS: ldur x0, [x1, #-257] ; CHECK-ERRORS: ^ @@ -93,16 +93,16 @@ foo: sqrshrn v7.4h, v8.4s, #39 uqshrn2 v4.4s, v5.2d, #67 -; CHECK-ERRORS: error: immediate must be an integer in range [1,8]. +; CHECK-ERRORS: error: immediate must be an integer in range [1, 8]. ; CHECK-ERRORS: sqrshrn b4, h9, #10 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: immediate must be an integer in range [1,8]. +; CHECK-ERRORS: error: immediate must be an integer in range [1, 8]. ; CHECK-ERRORS: rshrn v9.8b, v11.8h, #17 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: immediate must be an integer in range [1,16]. +; CHECK-ERRORS: error: immediate must be an integer in range [1, 16]. ; CHECK-ERRORS: sqrshrn v7.4h, v8.4s, #39 ; CHECK-ERRORS: ^ -; CHECK-ERRORS: error: immediate must be an integer in range [1,32]. +; CHECK-ERRORS: error: immediate must be an integer in range [1, 32]. ; CHECK-ERRORS: uqshrn2 v4.4s, v5.2d, #67 ; CHECK-ERRORS: ^ diff --git a/test/MC/ARM64/spsel-sysreg.s b/test/MC/ARM64/spsel-sysreg.s index fe556593afa..f1d94d8c2d8 100644 --- a/test/MC/ARM64/spsel-sysreg.s +++ b/test/MC/ARM64/spsel-sysreg.s @@ -19,6 +19,6 @@ mrs x0, ESR_EL1 msr DAIFSet, x0 msr ESR_EL1, #0 mrs x0, DAIFSet -// CHECK-ERRORS: error: expected writable system register or pstate +// CHECK-ERRORS: error: immediate must be an integer in range [0, 15] // CHECK-ERRORS: error: invalid operand for instruction // CHECK-ERRORS: error: expected readable system register