[Mlir-commits] [mlir] [mlir][spirv] Add bfloat16 support (PR #141458)
Igor Wodiany
llvmlistbot at llvm.org
Thu May 29 03:21:22 PDT 2025
================
@@ -866,11 +866,12 @@ LogicalResult spirv::Deserializer::processType(spirv::Opcode opcode,
typeMap[operands[0]] = IntegerType::get(context, operands[1], sign);
} break;
case spirv::Opcode::OpTypeFloat: {
- if (operands.size() != 2)
+ if (operands.size() < 2)
----------------
IgWod-IMG wrote:
What about making the check stronger here by only allowing `operands.size()` of 2 and 3. I don't think [spec](https://registry.khronos.org/SPIR-V/specs/unified1/SPIRV.html#OpTypeFloat) allows any other number of operands. The message below also should be changed - maybe: `"OpTypeFloat must have bitwidth parameter and optional floating point encoding"`.
https://github.com/llvm/llvm-project/pull/141458
More information about the Mlir-commits
mailing list