[Mlir-commits] [mlir] [mlir][spirv] Add support for structs decorations (PR #149793)

Igor Wodiany llvmlistbot at llvm.org
Tue Jul 22 05:17:41 PDT 2025


================
@@ -767,17 +769,50 @@ static Type parseStructType(SPIRVDialect const &dialect,
     return Type();
   }
 
-  if (failed(parser.parseRParen()) || failed(parser.parseGreater()))
+  if (failed(parser.parseRParen()))
+    return Type();
+
+  SmallVector<StructType::StructDecorationInfo, 0> structDecorationInfo;
----------------
IgWod-IMG wrote:

Yes, I followed what was done for member decorations in the print function. Although now I realised that in the parse function they set it to 4 for members. Anyway, after thinking about, I have updated the value to 1, because I think that the `Block` decoration is not that uncommon; and I don't think it's likely to have more than 1 decoration.

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


More information about the Mlir-commits mailing list