================
@@ -6636,22 +6635,25 @@ def CIR_MatrixExtractOp : CIR_Op<"matrix.extract", [
     ```
     %rowIdx = cir.const #cir.int<1> : !u64i
     %columnIdx = cir.const #cir.int<2> : !u64i
-    %element = cir.matrix.extract %matrix[%rowIdx : !u64i] [%columnIdx : 
!u64i] 
+    %row = cir.matrix.extract %matrix[%rowIdx : !s32i] 
+      : !cir.matrix<3 x 3 x !cir.float>
+    %element = cir.matrix.extract %matrix[%rowIdx : !s32i] [%columnIdx : 
!s32i] 
       : !cir.matrix<3 x 3 x !cir.float>
     ```
   }];
 
   let arguments = (ins 
     CIR_MatrixType:$matrix,
     CIR_AnyFundamentalIntType:$rowIdx,
-    CIR_AnyFundamentalIntType:$columnIdx
+    Optional<CIR_AnyFundamentalIntType>:$columnIdx
----------------
erichkeane wrote:

Can we document what it means to be missing this field? 

https://github.com/llvm/llvm-project/pull/232079
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to