[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