[llvm] [AMDGPU] Change the representation of double literals in operands (PR #68740)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 02:10:17 PDT 2023


================
@@ -4297,7 +4301,18 @@ bool AMDGPUAsmParser::validateVOPLiteral(const MCInst &Inst,
       continue;
 
     if (MO.isImm() && !isInlineConstant(Inst, OpIdx)) {
-      uint32_t Value = static_cast<uint32_t>(MO.getImm());
+      uint64_t Value = static_cast<uint64_t>(MO.getImm());
+      bool IsFP = AMDGPU::isSISrcFPOperand(Desc, OpIdx);
----------------
jayfoad wrote:

I don't fully understand this either. Are you trying to allow cases where the same 32-bit literal is used both as the high 32 bits of a double, and as a normal 32 bit integer, in the same instruction?

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


More information about the llvm-commits mailing list