[Mlir-commits] [mlir] [mlir] add tensor_static.extract/insert to take only static indices. (PR #110550)

donald chen llvmlistbot at llvm.org
Wed Oct 2 07:01:55 PDT 2024


cxy-1993 wrote:

After reviewing the previous discussions, I agree with @joker-eph  's point that when choosing between SSA values and attributes as operands for an op, the decision should be based on whether it is necessary for lowering. Given this conclusion, splitting ops into dynamic and static variants won't solve our current problem: when selecting an op for building, we should choose based on whether the lowering pattern supports dynamic behavior, not whether the parameters are static. Furthermore, if we were to adopt this approach, should we also refactor other existing ops like memref.subview?

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


More information about the Mlir-commits mailing list