[Mlir-commits] [mlir] [mlir][spirv] Add support for SPV_ARM_tensors (PR #144667)

[Igor Wodiany]

================
@@ -138,6 +138,7 @@ LogicalResult spirv::Deserializer::processHeader() {
       MIN_VERSION_CASE(3);
       MIN_VERSION_CASE(4);
       MIN_VERSION_CASE(5);
+      MIN_VERSION_CASE(6);
----------------
IgWod-IMG wrote:

I'll defer it to @kuhar. My main reason behind having it as a separate PR is that it gives us an opportunity to consider any implications of making this change. It's probably a trivial change, so there are no implications; but could there be any assumptions in the code that are not true for SPIR-V 1.6? I'm probably overthinking it, but I err on the side of caution.

Btw, the spec is written against 1.6, but the minimum version required is 1.0:

> This extension is written against the SPIR-V Specification, Version 1.6 Revision 2.

> This extension requires SPIR-V 1.0.

Source: https://github.com/KhronosGroup/SPIRV-Registry/blob/main/extensions/ARM/SPV_ARM_tensors.asciidoc


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