-
Notifications
You must be signed in to change notification settings - Fork 551
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 swizziling using transform dialect #16248
Add swizziling using transform dialect #16248
Conversation
6e67f25
to
8119319
Compare
8119319
to
fbddd0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
I don't see any IREE specific logics. Shouldn't we add the transform op upstream and use it in IREE? Do we plan to upstream them later? |
Yeah good point. @harsh-nod and @erman-gurses can we add this upstream? |
Can you not just run the pass directly? https://mlir.llvm.org/docs/Dialects/Transform/#transformapply_registered_pass-transformapplyregisteredpassop |
@MaheshRavishankar, @hanhanW, @Groverkss, I will look for a way to do that. |
Two PRs are landed to LLVM upstream regarding this PR. |
It connects the swizzling algorithm in the upstream LLVM with IREE using transform dialect.
See the LLVM upstream implementation: llvm/llvm-project#79164