[PATCH] D127609: [AtomicExpand] Make floating point conversion happens before fence insertion

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 12 21:12:00 PDT 2022


lkail created this revision.
lkail added reviewers: efriedma, jyknight, nemanjai, reames, PowerPC, arsenm.
Herald added subscribers: steven.zhang, hiraditya.
Herald added a project: All.
lkail requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

IIUC, the conversion part is not part of atomic operations. Fences should be put around converted atomic operations.
This also fixes atomic load of floating point values which requires fence on PowerPC.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D127609

Files:
  llvm/lib/CodeGen/AtomicExpandPass.cpp
  llvm/test/CodeGen/PowerPC/cfence-double.ll
  llvm/test/CodeGen/PowerPC/cfence-float.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D127609.436268.patch
Type: text/x-patch
Size: 7560 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220613/d9c3341b/attachment.bin>


More information about the llvm-commits mailing list