This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Don't always execute SC for min/max masked AMOs
AbandonedPublic

Authored by jrtc27 on Jan 20 2020, 1:11 PM.

Details

Reviewers
asb
lenary
Summary

If we determine from the comparison that no change is needed, there is
no point in us executing the SC and conditionally retrying, since the
AMO can be viewed as having executed atomically at the time of the LR.

Event Timeline

jrtc27 created this revision.Jan 20 2020, 1:11 PM
jrtc27 abandoned this revision.Jan 20 2020, 1:18 PM

Never mind, I see why this is done, we need to respect the ordering, so skipping the SC misses the release.