[PATCH] D135525: [LoongArch] Add codegen support for atomicrmw umin/umax operation on LA64

[Gong LingQin via Phabricator via llvm-commits]

gonglingqin added inline comments.


================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw-minmax.ll:76
+; LA64:       # %bb.0:
+; LA64-NEXT:    ammax_db.wu $a0, $a1, $a0
+; LA64-NEXT:    ret
----------------
SixWeining wrote:
> rd == rj ? Maybe we miss `earlyclobber`?
Thanks, I will rebase this patch.


================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw-minmax.ll:85
+; LA64:       # %bb.0:
+; LA64-NEXT:    ammax_db.du $a0, $a1, $a0
+; LA64-NEXT:    ret
----------------
SixWeining wrote:
> Ditto.
Thanks, I will rebase this patch.


================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw-minmax.ll:161
+; LA64:       # %bb.0:
+; LA64-NEXT:    ammin_db.wu $a0, $a1, $a0
+; LA64-NEXT:    ret
----------------
SixWeining wrote:
> Ditto.
Thanks, I will rebase this patch.


================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw-minmax.ll:170
+; LA64:       # %bb.0:
+; LA64-NEXT:    ammin_db.du $a0, $a1, $a0
+; LA64-NEXT:    ret
----------------
SixWeining wrote:
> Ditto.
Thanks, I will rebase this patch.


================
Comment at: llvm/test/CodeGen/LoongArch/ir-instruction/atomicrmw.ll:23
 ; LA32-NEXT:    sc.w $a5, $a2, 0
-; LA32-NEXT:    beq $a5, $zero, .LBB0_1
+; LA32-NEXT:    beqz $a5, .LBB0_1
 ; LA32-NEXT:  # %bb.2:
----------------
SixWeining wrote:
> Had better mention this change in summary.
Thanks, I will update the summary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135525