This is an archive of the discontinued LLVM Phabricator instance.

Use index for shape.to_extent_tensor
ClosedPublic

Authored by silvas on Apr 14 2020, 1:31 PM.

Details

Summary

Also,

  • add IndexTensor to OpBase.td
  • fix typo in the op name. It was mistakenly to_tensor instead of to_extent_tensor.

Diff Detail

Event Timeline

silvas created this revision.Apr 14 2020, 1:31 PM
rriddle accepted this revision.Apr 14 2020, 1:38 PM
rriddle added inline comments.
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
184

This seems to hint at a lack of coverage/usage in general.

This revision is now accepted and ready to land.Apr 14 2020, 1:38 PM
silvas marked an inline comment as done.Apr 14 2020, 1:57 PM
silvas added inline comments.
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
184

That's exactly why I'm pushing for the TCF/TCP/npcomp stuff. Otherwise we'll always be in this awkward dance where we are incrementally adding random stuff here based on out-of-tree usage.

This revision was automatically updated to reflect the committed changes.