[Mlir-commits] [mlir] [mlir][tensor] Remove assertion in ExpandShapeOp::build (PR #91361)

Han-Chung Wang llvmlistbot at llvm.org
Tue May 7 10:51:40 PDT 2024


hanhanW wrote:

I think we can capture the invalid op in op verifier. I'd suggest that we update the code below code as well. It should look at expand_shape shape inference as well. Then we can add the invalid op test to https://github.com/llvm/llvm-project/blob/main/mlir/test/Dialect/Tensor/invalid.mlir . What do you think?

https://github.com/llvm/llvm-project/blob/026a29e8b38aad79568de033d0e8e5d2e6bb4250/mlir/lib/Dialect/Tensor/IR/TensorOps.cpp#L1751-L1786

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


More information about the Mlir-commits mailing list