This is an archive of the discontinued LLVM Phabricator instance.

[mlir][python] Add Python bindings for ml_program dialect.
ClosedPublic

Authored by stellaraccident on May 17 2022, 10:43 PM.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 17 2022, 10:43 PM
stellaraccident requested review of this revision.May 17 2022, 10:43 PM

Update bazel files.

jpienaar accepted this revision.May 18 2022, 4:23 AM

LG, thanks

mlir/test/python/dialects/ml_program.py
9

(not required for here) We should probably make this a helper function given we duplicate this in many files :-)

20

Could we have one that modifies arg and res attrs?

This revision is now accepted and ready to land.May 18 2022, 4:23 AM
mlir/test/python/dialects/ml_program.py
9

I think I'm on with the duplication and personally like that the test files are standalone. But to probe a bit more, where do you think we should put it? (, This is not a trivial question because python scripts don't form a package)

stellaraccident marked an inline comment as done.May 18 2022, 11:01 PM
This revision was landed with ongoing or failed builds.May 18 2022, 11:09 PM
This revision was automatically updated to reflect the committed changes.