This patch also implements not emit fence in atomic binary operation
when AtomicOrdering is monotonic and fixes the issue of loading from
non ptr parameters.
The processing of other levels of AtomicOrdering will be added later.
Paths
| Differential D138481
[LoongArch] Add atomic ordering information for binary atomic operations ClosedPublic Authored by gonglingqin on Nov 22 2022, 1:24 AM.
Details Summary This patch also implements not emit fence in atomic binary operation The processing of other levels of AtomicOrdering will be added later.
Diff Detail
Unit TestsFailed
Event TimelineComment Actions Mostly LGTM, I was considering this optimization just before seeing this diff. But why will this change affect register allocation?
XiaodongLoong added a child revision: D138469: [LoongArch] Use tablegen size for getInstSizeInBytes.Nov 22 2022, 11:31 PM This revision is now accepted and ready to land.Nov 23 2022, 2:11 AM This revision was landed with ongoing or failed builds.Nov 28 2022, 4:08 AM Closed by commit rGa2d10bda1870: [LoongArch] Add atomic ordering information for binary atomic operations (authored by gonglingqin). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 477103 llvm/lib/Target/LoongArch/LoongArchExpandAtomicPseudoInsts.cpp
llvm/lib/Target/LoongArch/LoongArchInstrInfo.td
llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw.ll
|
I mean, why this is changed?