From dbf0d8118ccbfd09def0974bf671c39e94d0be93 Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Fri, 27 Aug 2021 12:30:31 -0700 Subject: [PATCH] [RISCV] Use ~0ULL instead of ~0U when checking for invalid ErrorInfo. ErrorInfo is a uint64_t and is initialized to all 1s. Not sure how to test this. Noticed while working on .insn support. --- llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp b/llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp index b2683d575e14..c3ccec10d804 100644 --- a/llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp +++ b/llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp @@ -1002,7 +1002,7 @@ bool RISCVAsmParser::MatchAndEmitInstruction(SMLoc IDLoc, unsigned &Opcode, } case Match_InvalidOperand: { SMLoc ErrorLoc = IDLoc; - if (ErrorInfo != ~0U) { + if (ErrorInfo != ~0ULL) { if (ErrorInfo >= Operands.size()) return Error(ErrorLoc, "too few operands for instruction"); @@ -1019,7 +1019,7 @@ bool RISCVAsmParser::MatchAndEmitInstruction(SMLoc IDLoc, unsigned &Opcode, // corresponding operand is missing. if (Result > FIRST_TARGET_MATCH_RESULT_TY) { SMLoc ErrorLoc = IDLoc; - if (ErrorInfo != ~0U && ErrorInfo >= Operands.size()) + if (ErrorInfo != ~0ULL && ErrorInfo >= Operands.size()) return Error(ErrorLoc, "too few operands for instruction"); }