[PATCH] D110069: AArch64: use `CAS` instead of `LDX`/`STX` for more ops if available
Tim Northover via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 13 07:11:06 PST 2022
t.p.northover marked 2 inline comments as done.
t.p.northover added inline comments.
================
Comment at: llvm/test/CodeGen/AArch64/atomicrmw-xchg-fp.ll:107
+; LSE-NEXT: mov x4, x6
+; LSE-NEXT: mov x5, x7
+; LSE-NEXT: caspal x4, x5, x2, x3, [x0]
----------------
efriedma wrote:
> These moves seem very strange.
The first two are part of forming an `xseqregclass` thing from component registers, the second two are because `CASP` clobbers its input but we want to compare against it afterwards. Still not ideal, but not completely out there.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110069/new/
https://reviews.llvm.org/D110069
More information about the llvm-commits
mailing list