[Mlir-commits] [mlir] [AMDGPU] Change the representation of double literals in operands (PR #68740)

Jay Foad llvmlistbot at llvm.org
Thu Oct 12 02:16:38 PDT 2023


================
@@ -454,6 +454,8 @@ void AMDGPUInstPrinter::printImmediate64(uint64_t Imm,
   else if (Imm == 0x3fc45f306dc9c882 &&
            STI.hasFeature(AMDGPU::FeatureInv2PiInlineImm))
     O << "0.15915494309189532";
+  else if (IsFP && AMDGPU::isValid32BitLiteral(Imm, true))
----------------
jayfoad wrote:

Can this be:
```suggestion
  else if (IsFP) {
    assert(AMDGPU::isValid32BitLiteral(Imm, true));
```

https://github.com/llvm/llvm-project/pull/68740


More information about the Mlir-commits mailing list