[PATCH] D130191: [RISCV] Teach ComputeNumSignBitsForTargetNode about Intrinsic::riscv_masked_cmpxchg_i64

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 11:56:11 PDT 2022


jrtc27 added a comment.

Do we care about other masked i64 atomic intrinsics, and do the i32 ones ever see a benefit from implementing an equivalent?



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:9679
+      break;
+    case Intrinsic::riscv_masked_cmpxchg_i64:
+      // The output of the intrinsic is generated by performing an LR_W and
----------------
Should have extra { } so it's easy to add other cases


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

https://reviews.llvm.org/D130191



More information about the llvm-commits mailing list