-
Notifications
You must be signed in to change notification settings - Fork 31
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
Loop tiling, shuffle and expansion passes #922
Conversation
I haven't done a detailed review, but it seems the old 2d parallelization is disabled/removed from tpp-run. I suggest for now we keep the old one functional in the default pass and make the new one available through a different knob. Then we can compare both and retire the old one, when the new one completely covers the old usecases. |
7a1aa5d
to
e6c1086
Compare
I am working on unit tests for the passes as well as MLP integration tests, in the meantime we can start the review. |
799e90f
to
16cea04
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.
Initial general comments.
ca7692b
to
c1ce01e
Compare
c1ce01e
to
73d293f
Compare
… the shape of the tensor
86d94e0
to
9a0ab09
Compare
Closing this PR since we're moving this pass to vector dialect. |
No description provided.