[PATCH] D135948: [LoongArch] Add codegen support for cmpxchg on LA64

Fri Oct 14 02:00:57 PDT 2022

xry111 added inline comments.

================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomic-cmpxchg.ll:20
+; LA64-NEXT:  .LBB0_1: # =>This Inner Loop Header: Depth=1
+; LA64-NEXT:    dbar 0
+; LA64-NEXT:    ll.w $a4, $a3, 0
----------------
hev wrote:
> Not needed.
> 
> ll: full-membar + load-exclusive
Jiaxun told me (via linux-mips):

> I had checked with Loongson guys and they confirmed that the workaround still needs to be applied to latest 3A4000
> processors, including 3A4000 for MIPS and 3A5000 for LoongArch. Though, the reason behind the workaround varies with the evaluation
> of  their uArch, for GS464V based core, barrier is required as the uArch design allows regular load to be reordered after an atomic linked
> load, and that would break  assumption of compiler atomic constraints.

In GCC we use `dbar 0x700`, so in the future HW engineers can fix this issue and make `dbar 0x700` no-op.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135948/new/

https://reviews.llvm.org/D135948