[PATCH] D73066: [RISCV] Don't always execute SC for min/max masked AMOs

James Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 20 13:13:50 PST 2020


jrtc27 created this revision.
jrtc27 added reviewers: asb, lenary.
Herald added subscribers: llvm-commits, luismarques, apazos, sameer.abuasal, pzheng, s.egerton, Jim, benna, psnobl, jocewei, PkmX, jfb, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, MaskRay, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, hiraditya.
Herald added a project: LLVM.

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.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D73066

Files:
  llvm/lib/Target/RISCV/RISCVExpandPseudoInsts.cpp
  llvm/test/CodeGen/RISCV/atomic-rmw.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73066.239195.patch
Type: text/x-patch
Size: 50385 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200120/0e68a4de/attachment-0001.bin>


More information about the llvm-commits mailing list