Skip to content

Revert "[NVPTX] Lower 16xi8 and 8xi8 stores efficiently (#73646)"#74518

Merged
Artem-B merged 1 commit intollvm:mainfrom
Artem-B:revert-173fcf7
Dec 6, 2023
Merged

Revert "[NVPTX] Lower 16xi8 and 8xi8 stores efficiently (#73646)"#74518
Artem-B merged 1 commit intollvm:mainfrom
Artem-B:revert-173fcf7

Conversation

@Artem-B
Copy link
Member

@Artem-B Artem-B commented Dec 5, 2023

This reverts commit 173fcf7.

Needs to constrain the optimization to properly aligned loads/stores only. #73646 (comment)

This reverts commit 173fcf7.

Needs to constrain the optimization to properly aligned loads/stores only.
llvm#73646 (comment)
@Artem-B
Copy link
Member Author

Artem-B commented Dec 5, 2023

@bondhugula FYI.

@Artem-B
Copy link
Member Author

Artem-B commented Dec 5, 2023

I'll hold off on revert until tomorrow, so @bondhugula may have a chance to fix it forward.

@steven-johnson
Copy link

Any update on this?

@Artem-B Artem-B merged commit a2d3bb1 into llvm:main Dec 6, 2023
@Artem-B Artem-B deleted the revert-173fcf7 branch December 6, 2023 18:49
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