[PATCH] D81733: GlobalISel: Don't fail translate on weak cmpxchg

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 09:15:05 PDT 2020


jfb added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll:1980
+; CHECK-NEXT:    [[NEWVAL:%[0-9]+]]:_(s32) = G_CONSTANT i32 1
+; CHECK:       bb.2.repeat:
+; CHECK-NEXT:    successors: %bb.3({{[^)]+}}), %bb.2({{[^)]+}})
----------------
arsenm wrote:
> jfb wrote:
> > Does this now contains two loops instead of just one?
> No? Not sure why you think there is
The label name is different. It's a fundamental property of weak cmpxchg that it's (mostly) used inside a user-provided loop, so it's important to make sure we don't add an inner loop when translating it.


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

https://reviews.llvm.org/D81733





More information about the llvm-commits mailing list