[Mlir-commits] [mlir] [mlir][Hoisting] Hoisting vector.extract/vector.broadcast pairs (PR #86108)

[Steven Varoumas]

================
@@ -43,6 +43,120 @@ using llvm::dbgs;
 using namespace mlir;
 using namespace mlir::linalg;
 
+scf::ForOp replaceWithDifferentYield(RewriterBase &rewriter, scf::ForOp loop,
----------------
stevenvar wrote:

Oh yeah I thought about this. The main issue I have with it is that it would change the ordering of parameters (if there is more than one yielded value), which could make the pass confusing. Do you think it is preferable anyway to introducing `replaceWithDifferentYield`? 

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