Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add note about MLIR compiled outputs in dev docs #1195

Merged
merged 1 commit into from
Aug 10, 2022
Merged

Conversation

rengolin
Copy link
Member

@rengolin rengolin commented Aug 9, 2022

Adding an example on how to extract MLIR output from the compilation
process in various different formats to the development documentation.

This should help developers trying to either debug torch_mlir or use it
for the purpose of extracting MLIR outputs for other testing.

Fixes #1175

@rengolin rengolin requested a review from silvasean August 9, 2022 10:26
Copy link
Collaborator

@powderluv powderluv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un-intended mhlo update ?

Adding an example on how to extract MLIR output from the compilation
process in various different formats to the development documentation.

This should help developers trying to either debug torch_mlir or use it
for the purpose of extracting MLIR outputs for other testing.

Fixes llvm#1175
@rengolin
Copy link
Member Author

rengolin commented Aug 9, 2022

Un-intended mhlo update ?

Argh, sorry! I built in-tree, probably changed something? Update now.

@rengolin rengolin merged commit 71c240a into llvm:main Aug 10, 2022
@silvasean
Copy link
Contributor

Thanks for the PR!

qedawkins pushed a commit to nod-ai/torch-mlir that referenced this pull request Oct 3, 2022
Several properties (such as verifier, printer & parser) are going to be obsolete in main mlir soon. This change removes the explicit usage of verifier and instead uses the hasVerifier option instead.

Also, some minor whitespace cleanup.

Signed-off-by: Stella Stamenova stilis@microsoft.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using Torch-MLIR as a front-end for MLIR files
3 participants