This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Select i8/i16 global and flat atomic load/store
ClosedPublic

Authored by arsenm on Apr 12 2022, 3:35 PM.

Details

Reviewers
t-tye
AlexVlx
Group Reviewers
Restricted Project
Summary

As far as I know these should be atomic anyway, as long as the address
is aligned. Unaligned atomics hit an ugly error in AtomicExpand.

Diff Detail

Event Timeline

arsenm created this revision.Apr 12 2022, 3:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2022, 3:35 PM
arsenm requested review of this revision.Apr 12 2022, 3:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2022, 3:35 PM
Herald added a subscriber: wdng. · View Herald Transcript
AlexVlx accepted this revision as: AlexVlx.Apr 14 2022, 1:12 AM

LGTM, thanks (the build failure is yet again weird and unrelated AFAICT).

This revision is now accepted and ready to land.Apr 14 2022, 1:12 AM