Skip to content

Update MLIR package with fix for performance improvement of one-shot-bufferize#83

Merged
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/mlir-bufferize
Jun 2, 2026
Merged

Update MLIR package with fix for performance improvement of one-shot-bufferize#83
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/mlir-bufferize

Conversation

@guillon
Copy link
Copy Markdown
Member

@guillon guillon commented Jun 1, 2026

Motivation

Update dependencies to last xtc-mlir packages which include the performance improvement for one-shot-bufferize.

The slow implementation of bufferization was oberved by @liamsemeria related to the PR #45.

Actually a fix was contributed to upstream MLIR with this PR llvm/llvm-project#189895

We integrated the modification as a patch on the currently packaged xtc-mlir-wheels on top of LLVM/MLIR revision 21.1.2: xtc-tools/xtc-mlir-wheels@0471d95

Description

Simply update package revisions to the corresponding xtc-mlir pypi package versions.

Commits

Note that there is also an unrelated commit to add the covergae badge to the XTC Readme.

Discussion

To be validated in term of performance improvement in PR #45

TODO

@guillon
Copy link
Copy Markdown
Member Author

guillon commented Jun 1, 2026

@liamsemeria Can you confirm the expected performance improvement when cherry-picking commit "mlir: update mlir packages to to 21.1.2.8" within your branch?

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@guillon guillon added the enhancement New feature or request label Jun 1, 2026
guillon added 2 commits June 2, 2026 10:32
This package version fixes performance issues with
MLIR one shot bufferize.
@guillon guillon force-pushed the dev/cguillon/mlir-bufferize branch from 1283ad2 to 3815438 Compare June 2, 2026 08:33
@liamsemeria
Copy link
Copy Markdown
Contributor

@liamsemeria Can you confirm the expected performance improvement when cherry-picking commit "mlir: update mlir packages to to 21.1.2.8" within your branch?
@guillon with the new commit loop-explore with the tensor dialect takes around 1.5x longer than memref and before it was around 6x longer so I think the patch is working. I was expecting it to be a bit faster but its still good.

@guillon
Copy link
Copy Markdown
Member Author

guillon commented Jun 2, 2026

@liamsemeria Can you confirm the expected performance improvement when cherry-picking commit "mlir: update mlir packages to to 21.1.2.8" within your branch?
@guillon with the new commit loop-explore with the tensor dialect takes around 1.5x longer than memref and before it was around 6x longer so I think the patch is working. I was expecting it to be a bit faster but its still good.

ok thansk for the check

@guillon guillon assigned guillon and unassigned liamsemeria Jun 2, 2026
@guillon guillon merged commit 70d6b13 into xtc-tools:main Jun 2, 2026
3 checks passed
@guillon guillon deleted the dev/cguillon/mlir-bufferize branch June 2, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants