This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add intrinsics for alignbit and alignbyte instructions
ClosedPublic

Authored by rampitec on Jun 8 2017, 4:49 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

rampitec created this revision.Jun 8 2017, 4:49 PM
b-sumner edited edge metadata.Jun 9 2017, 7:27 AM

This looks fine to me. I was concerned about how successful we would be pattern matching them.

This looks fine to me. I was concerned about how successful we would be pattern matching them.

We do not match it after numerous of optimizations, it is better to use intrinsic from the library.

arsenm accepted this revision.Jun 9 2017, 11:48 AM

LGTM. We should try to pattern match these though, I think through ROTR one is matched sometimes already

This revision is now accepted and ready to land.Jun 9 2017, 11:48 AM
This revision was automatically updated to reflect the committed changes.