[PATCH] D106039: [AArch64] Fix i128 cmpxchg using ldxp/stxp.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 20 13:24:32 PDT 2021


efriedma added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:17058
 
+  unsigned Opcode;
+  switch (MemOp->getMergedOrdering()) {
----------------
MaskRay wrote:
> This was unused.  I deleted it in 2b79a4eeb101bc9662427353187e35b9bc9da116 to pacify -Wunused-but-set-variable
Fixed it to do what I meant to do in 664a1fd9f04de65f0a37d612845dc9e095abe067

@samparker Let me know if you want me to add more test coverage.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106039



More information about the llvm-commits mailing list