check for 32-bit immediate instead

llvm · Oct 15, 2024 · 665720f · 665720f
1 parent 9679519
commit 665720f
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp b/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
@@ -1158,10 +1158,9 @@ class ARMOperand : public MCParsedAsmOperand {
   bool isFPImm() const {
     if (!isImm()) return false;
     const MCConstantExpr *CE = dyn_cast<MCConstantExpr>(getImm());
-    if (!CE) return false;
-    // TODO: Is implicitTrunc correct here?
-    int Val = ARM_AM::getFP32Imm(
-        APInt(32, CE->getValue(), /*isSigned=*/true, /*implicitTrunc=*/true));
+    if (!CE || !isUInt<32>(CE->getValue()))
+      return false;
+    int Val = ARM_AM::getFP32Imm(APInt(32, CE->getValue()));
     return Val != -1;
   }