[Mlir-commits] [mlir] [MLIR][Linalg] Use Top-Down traversal to safely optimize multi-use producer fusion (PR #172216)

[Miloš Poletanović]

milos1397 wrote:

Hey @MaheshRavishankar, thanks for the review.

That’s correct, this PR just fixes the test case. In this instance, we are preparing the data so that `linalg::fuseElementwiseOps` processes operations in an order that yields a valid result (as explained above).
Essentially, we can establish that a current precondition for using` linalg::fuseElementwiseOps` is that the IR must be processed in a **TopDown** order.
On the other side, more robust logic could be added to `linalg::fuseElementwiseOps` itself to handle these cases automatically and prevent the ordering issue described above.

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