================
@@ -630,7 +639,10 @@ class FlattenContiguousRowMajorTransferReadPattern
     if (transferReadOp.getMask())
       return failure();
 
-    int64_t firstDimToCollapse = sourceType.getRank() - vectorType.getRank();
----------------
newling wrote:

Why does this need to change? 

If memref is rank n+2 and vector is rank n, isn't it always fine to flatten the 
memref from index 2? So that memref becomes rank 3 and vector becomes rank 1. 
Isn't having a rank-1 vector the goal here? 


https://github.com/llvm/llvm-project/pull/142422
_______________________________________________
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