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

Loop tiling, shuffle and expansion passes #922

Closed
wants to merge 36 commits into from

Conversation

KavithaTipturMadhu
Copy link
Contributor

No description provided.

@alheinecke
Copy link
Contributor

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.

@KavithaTipturMadhu KavithaTipturMadhu force-pushed the looptiling branch 10 times, most recently from 7a1aa5d to e6c1086 Compare June 12, 2024 07:25
@KavithaTipturMadhu KavithaTipturMadhu marked this pull request as ready for review June 12, 2024 08:29
@KavithaTipturMadhu
Copy link
Contributor Author

I am working on unit tests for the passes as well as MLP integration tests, in the meantime we can start the review.

@rengolin rengolin added the benchmark Triggers benchmark jobs label Jun 12, 2024
@KavithaTipturMadhu KavithaTipturMadhu force-pushed the looptiling branch 4 times, most recently from 799e90f to 16cea04 Compare June 12, 2024 12:37
Copy link
Collaborator

@adam-smnk adam-smnk left a comment

Choose a reason for hiding this comment

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

Initial general comments.

lib/TPP/DefaultPipeline.cpp Outdated Show resolved Hide resolved
include/TPP/Passes.td Outdated Show resolved Hide resolved
include/TPP/Passes.td Outdated Show resolved Hide resolved
lib/TPP/Transforms/LoopInsertion.cpp Outdated Show resolved Hide resolved
lib/TPP/Transforms/LoopInsertion.cpp Outdated Show resolved Hide resolved
lib/TPP/Transforms/LoopExpansion.cpp Outdated Show resolved Hide resolved
lib/TPP/Transforms/LoopInsertion.cpp Outdated Show resolved Hide resolved
@KavithaTipturMadhu KavithaTipturMadhu force-pushed the looptiling branch 4 times, most recently from ca7692b to c1ce01e Compare June 15, 2024 06:39
@KavithaTipturMadhu KavithaTipturMadhu added benchmark Triggers benchmark jobs and removed benchmark Triggers benchmark jobs labels Jun 15, 2024
@KavithaTipturMadhu KavithaTipturMadhu removed the benchmark Triggers benchmark jobs label Jun 15, 2024
@KavithaTipturMadhu KavithaTipturMadhu added benchmark Triggers benchmark jobs and removed benchmark Triggers benchmark jobs labels Jul 31, 2024
@KavithaTipturMadhu
Copy link
Contributor Author

Closing this PR since we're moving this pass to vector dialect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmark Triggers benchmark jobs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants