[Mlir-commits] [mlir] [mlir][TilingInterface] Extend option to yield replacement for multiple results case (PR #93144)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Mon Jun 3 23:08:33 PDT 2024


================
@@ -190,10 +190,14 @@ tileAndFuseProducerOfSlice(RewriterBase &rewriter,
 /// where `%0` had other uses as well. If not reconstructed from within the loop
 /// body, uses of `%0` could not be replaced, making it still live and the
 /// fusion immaterial.
+///
+/// The @param `yieldResultNumber` decides which result would be yield. If not
+/// given, yield all `opResult` of fused producer.
----------------
MaheshRavishankar wrote:

Could you make this more clear. I always find `std::optional<ArrayRef<unsigned>>` to be a bit ambiguous. What does it mean if we pass an `ArrayRef<unsigned>{}`?

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


More information about the Mlir-commits mailing list