It looks like it was a typo. Instead of *maybeConstantIndex,
initTensorOp.getStaticSize(*maybeConstantIndex) should be used to access the
dim size of the tensor. There is a test for that in canonicalize.mlir, but it
was working correctly because ReplaceStaticShapeDims was canonicalizing DimOp
before FoldInitTensorWithDimOp. So, to make the patterns more "orthogonal",
this case is disabled.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo