This is an archive of the discontinued LLVM Phabricator instance.

[Mlir] Add assembly format for `shape.mul`.
ClosedPublic

Authored by pifon2a on Jun 4 2020, 1:41 PM.

Diff Detail

Event Timeline

pifon2a created this revision.Jun 4 2020, 1:41 PM
Herald added a reviewer: silvas. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
silvas accepted this revision.Jun 4 2020, 1:43 PM
silvas added inline comments.
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
285

FYI, I think after Jacques' recent patch, you can remove DeclareOpInterfaceMethods<InferTypeOpInterface> as well. I think it would be cool to include that in this patch as well.

jpienaar accepted this revision.Jun 4 2020, 1:48 PM
jpienaar added inline comments.
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
285

I thought I removed all of these earlier this week ...

This revision is now accepted and ready to land.Jun 4 2020, 1:48 PM
pifon2a marked 3 inline comments as done.Jun 5 2020, 1:59 AM
pifon2a added inline comments.
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
285

it was removed by Jacques. I just needed to rebase...

This revision was automatically updated to reflect the committed changes.
pifon2a marked an inline comment as done.