This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] SILoadStoreOptimizer: Remove redundant check for volatile
ClosedPublic

Authored by foad on Jan 26 2022, 9:39 AM.

Details

Summary

SILoadStoreOptimizer::collectMergeableInsts already ends the current
block if it sees a volatile (or ordered) memory access, so there is no
need to check for them again when scanning the instructions between two
pairing candidates in a block.

Diff Detail

Event Timeline

foad created this revision.Jan 26 2022, 9:39 AM
foad requested review of this revision.Jan 26 2022, 9:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2022, 9:39 AM
arsenm accepted this revision.Jan 26 2022, 9:43 AM
This revision is now accepted and ready to land.Jan 26 2022, 9:43 AM
This revision was landed with ongoing or failed builds.Jan 27 2022, 2:24 AM
This revision was automatically updated to reflect the committed changes.