This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add volatile support to SIMemoryLegalizer
ClosedPublic

Authored by t-tye on Jan 6 2021, 10:59 PM.

Details

Summary

Treat a non-atomic volatile load and store as a relaxed atomic at
system scope for the address spaces accessed. This will ensure all
relevant caches will be bypassed.

A volatile atomic is not changed and still only bypasses caches upto
the level specified by the SyncScope operand.

Diff Detail

Event Timeline

t-tye created this revision.Jan 6 2021, 10:59 PM
t-tye requested review of this revision.Jan 6 2021, 10:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 6 2021, 10:59 PM

LGTM. Make sure to rerun tests before push as a lot of tests are affected.

rampitec accepted this revision.Jan 8 2021, 12:02 PM
This revision is now accepted and ready to land.Jan 8 2021, 12:02 PM
This revision was landed with ongoing or failed builds.Jan 8 2021, 4:53 PM
This revision was automatically updated to reflect the committed changes.