[llvm] [RISCV] Merge int_riscv_masked_atomicrmw_*_i32/i64 intrinsics using llvm_anyint_ty. (PR #154845)
Sam Elliott via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 21 21:25:51 PDT 2025
================
@@ -1800,15 +1800,15 @@ bool RISCVTargetLowering::getTgtMemIntrinsic(IntrinsicInfo &Info,
switch (Intrinsic) {
default:
return false;
- case Intrinsic::riscv_masked_atomicrmw_xchg_i32:
- case Intrinsic::riscv_masked_atomicrmw_add_i32:
- case Intrinsic::riscv_masked_atomicrmw_sub_i32:
- case Intrinsic::riscv_masked_atomicrmw_nand_i32:
- case Intrinsic::riscv_masked_atomicrmw_max_i32:
- case Intrinsic::riscv_masked_atomicrmw_min_i32:
- case Intrinsic::riscv_masked_atomicrmw_umax_i32:
- case Intrinsic::riscv_masked_atomicrmw_umin_i32:
- case Intrinsic::riscv_masked_cmpxchg_i32:
+ case Intrinsic::riscv_masked_atomicrmw_xchg:
+ case Intrinsic::riscv_masked_atomicrmw_add:
+ case Intrinsic::riscv_masked_atomicrmw_sub:
+ case Intrinsic::riscv_masked_atomicrmw_nand:
+ case Intrinsic::riscv_masked_atomicrmw_max:
+ case Intrinsic::riscv_masked_atomicrmw_min:
+ case Intrinsic::riscv_masked_atomicrmw_umax:
+ case Intrinsic::riscv_masked_atomicrmw_umin:
+ case Intrinsic::riscv_masked_cmpxchg:
Info.opc = ISD::INTRINSIC_W_CHAIN;
----------------
lenary wrote:
Please can you add a comment to these cases, saying that the widest these are expanded to are 32-bits, and it's not easy to know any better than that as the mask size/shift is dynamic?
You sort-of have a similar comment below, in the next hunk down. I just don't want to have to re-discover this info from scratch.
https://github.com/llvm/llvm-project/pull/154845
More information about the llvm-commits
mailing list