[llvm] [mlir][vector] Enable transfer op hoisting with dynamic indices (PR #68500)

Lei Zhang via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 15 11:23:32 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 llvm-commits mailing list