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

Benoit Jacob llvmlistbot at llvm.org
Tue May 7 11:42:59 PDT 2024


bjacob wrote:

Yeah IR like this would go to the `ExpandShapeOp::build` overload that takes a `outputShape`.

To exercise the error condition we are talking about here, which is in the `ExpandShapeOp::build` overload NOT taking a `outputShape`, I think the only way to hit that is from C++. I don't see it being hit from any kind of tensor-dialect MLIR source, since there the `output_shape` is mandatory in the syntax, and then of course it's going to call the C++ overload with `outputShape` parameter.


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


More information about the Mlir-commits mailing list