Skip to content

Conversation

wenju-he
Copy link
Contributor

Also internalize _clc_get_el* symbols in clc_shuffle2. llvm-diff shows no change to amdgcn--amdhsa.bc.

Also internalize __clc_get_el_* symbols in clc_shuffle2.
llvm-diff shows no change to amdgcn--amdhsa.bc.
@wenju-he
Copy link
Contributor Author

@frasercrmck please help to review, thanks

Copy link
Contributor

@frasercrmck frasercrmck left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@frasercrmck
Copy link
Contributor

Sorry, just merged #140557 so you'll have to fix the conflicts before merging.

@wenju-he
Copy link
Contributor Author

Sorry, just merged #140557 so you'll have to fix the conflicts before merging.

done, thanks

@frasercrmck frasercrmck merged commit e70568e into llvm:main May 21, 2025
9 checks passed
@wenju-he wenju-he deleted the clc-shuffle2 branch June 5, 2025 08:19
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