[PATCH] D98085: [AMDGPU] Always expand system scope fp atomics on gfx90a
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 5 17:24:32 PST 2021
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:11954-11955
+ auto SSID = RMW->getSyncScopeID();
+ if (SSID == SyncScope::System ||
+ SSID == RMW->getContext().getOrInsertSyncScopeID("one-as"))
+ return AtomicExpansionKind::CmpXChg;
----------------
rampitec wrote:
> arsenm wrote:
> > Isn't there some scope greater or equal function? Should avoid directly referencing one-as
> It is, but it is located in the MMI whcih is not yet available here. In fact I don't even have MF yet to query it.
>
> Anyway since it is specific to the only target and that target does not have a higher scope I suppose it shall be OK.
Can it be moved out of the MMI? That sounds like a weird place for it
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98085/new/
https://reviews.llvm.org/D98085
More information about the llvm-commits
mailing list