[Mlir-commits] [mlir] Add support for MLIR to llvm vscale attribute (PR #67012)

Mats Petersson llvmlistbot at llvm.org
Fri Sep 22 11:25:16 PDT 2023


================
@@ -2342,6 +2342,19 @@ ParseResult LLVMFuncOp::parse(OpAsmParser &parser, OperationState &result) {
   result.addAttribute(getFunctionTypeAttrName(result.name),
                       TypeAttr::get(type));
 
+  if (succeeded(parser.parseOptionalKeyword("vscale_range"))) {
----------------
Leporacanthicus wrote:

Well, from what my experiments tell me, that won't work - if I have it there, tt needs to be an attribute of the function, that is read during parsing, because those are stored somewhere else than the ones given as "random attributes". 

Anyways, I will submit this on Moinday, unless I see any further comments that need fixing. I have technically already "gone home", but working from home, I have my computer in front of me after I've finished... :)

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


More information about the Mlir-commits mailing list