[Mlir-commits] [mlir] [mlir][vector] Make the in_bounds attribute mandatory (PR #97049)

Diego Caballero llvmlistbot at llvm.org
Thu Jul 11 09:34:55 PDT 2024


================
@@ -1223,8 +1223,19 @@ static Operation *vectorizeAffineLoad(AffineLoadOp loadOp,
   LLVM_DEBUG(dbgs() << "\n[early-vect]+++++ permutationMap: ");
   LLVM_DEBUG(permutationMap.print(dbgs()));
 
+  // Make sure that the in_bounds attribute corresponding to a broadcast dim
+  // is set to `true` - that's required by the xfer Op.
----------------
dcaballe wrote:

Nothing for this PR but this is still a constraint that seems unnecessary and makes this complicated, as we can see here.

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


More information about the Mlir-commits mailing list