[Mlir-commits] [mlir] [mlir][vector] Enable transfer op hoisting with dynamic indices (PR #68500)
Lei Zhang
llvmlistbot at llvm.org
Sun Oct 15 11:23:30 PDT 2023
antiagainst wrote:
> Btw, it seems that `testDynamicValueUsingBounds` is set to `true` in most places. Perhaps, instead, that should be controllable through the transform op? (i.e. `hoist_redundant_vector_transfers`)
The flag is meant to avoid bring in the overhead for things that should be fast, e.g., op folders and such. For hoisting and forwarding, I think it's fine to always enable so no need to expose the control.
https://github.com/llvm/llvm-project/pull/68500
More information about the Mlir-commits
mailing list