================
@@ -1064,16 +1064,15 @@ static void printMapClause(OpAsmPrinter &p, Operation 
*op,
 }
 
 static ParseResult parseMembersIndex(OpAsmParser &parser,
-                                     DenseIntElementsAttr &membersIdx) {
-  SmallVector<APInt> values;
+                                     ArrayAttr &membersIdx) {
+  SmallVector<Attribute, 4> values, memberIdxs;
----------------
skatrak wrote:

Nit: Is there a reason for the magic number "4" here? If not, it's generally 
preferred to leave the default (or zero, if it refuses to give you a 
compile-time default).

https://github.com/llvm/llvm-project/pull/96265
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to