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

Move byte manipulation ops from elwise ops conversion. #28

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

ienkovich
Copy link
Collaborator

We have many operation conversions in ConvertElementwiseOps pass and some of them are not actually elementeise. It started with a single pattern because it was not reasonable to introduce a new pass for it, but more and more operations were added later. This patch moves those patterns into a new pass that covers vector element manipulations, like shuffles, broadcasts, etc. This is pure refactoring, with no functional changes

@ienkovich ienkovich requested a review from minjang June 18, 2024 16:48
@ienkovich ienkovich requested a review from ptillet as a code owner June 18, 2024 16:48
Copy link
Collaborator

@minjang minjang left a comment

Choose a reason for hiding this comment

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

Thanks for the refactoring!

@minjang minjang merged commit 44035c9 into triton-lang:main Jun 20, 2024
2 of 4 checks passed
@ienkovich ienkovich deleted the ienkovich/cpu/elemwise-refactor branch June 20, 2024 21:26
minjang pushed a commit to minjang/triton-cpu that referenced this pull request Jun 22, 2024
minjang pushed a commit that referenced this pull request Jun 24, 2024
Devjiu pushed a commit to Devjiu/triton-cpu that referenced this pull request Aug 13, 2024
int3 pushed a commit that referenced this pull request Aug 29, 2024
minjang pushed a commit that referenced this pull request Sep 22, 2024
minjang pushed a commit that referenced this pull request Oct 22, 2024
minjang pushed a commit that referenced this pull request Oct 24, 2024
int3 pushed a commit that referenced this pull request Dec 6, 2024
ienkovich added a commit that referenced this pull request Dec 6, 2024
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.

2 participants