diff --git a/lib/Target/Mips/AsmParser/MipsAsmParser.cpp b/lib/Target/Mips/AsmParser/MipsAsmParser.cpp index 602bf04dac3..5bc8f9d4b63 100644 --- a/lib/Target/Mips/AsmParser/MipsAsmParser.cpp +++ b/lib/Target/Mips/AsmParser/MipsAsmParser.cpp @@ -1217,7 +1217,7 @@ unsigned MipsAsmParser::getGPR(int RegNo) { int MipsAsmParser::matchRegisterByNumber(unsigned RegNum, unsigned RegClass) { if (RegNum > - getContext().getRegisterInfo()->getRegClass(RegClass).getNumRegs()) + getContext().getRegisterInfo()->getRegClass(RegClass).getNumRegs() - 1) return -1; if (RegClass == Mips::GPR32RegClassID || RegClass == Mips::GPR64RegClassID) diff --git a/test/MC/Mips/mips-register-names-invalid.s b/test/MC/Mips/mips-register-names-invalid.s new file mode 100644 index 00000000000..df1054fed42 --- /dev/null +++ b/test/MC/Mips/mips-register-names-invalid.s @@ -0,0 +1,8 @@ +# RUN: not llvm-mc %s -triple=mips-unknown-freebsd -show-encoding 2>%t0 +# RUN: FileCheck %s < %t0 + +# $32 used to trigger an assertion instead of the usual error message due to +# an off-by-one bug. + +# CHECK: :[[@LINE+1]]:18: error: invalid operand for instruction + add $32, $0, $0