[llvm] [LoongArch] Introduce `32s` target feature for LA32S ISA extensions (PR #139695)

WÁNG Xuěruì via llvm-commits llvm-commits at lists.llvm.org
Tue May 13 05:21:24 PDT 2025


================
@@ -214,8 +214,9 @@ static void doAtomicBinOpExpansion(const LoongArchInstrInfo *TII,
       .addReg(ScratchReg)
       .addReg(AddrReg)
       .addImm(0);
-  BuildMI(LoopMBB, DL, TII->get(LoongArch::BEQZ))
+  BuildMI(LoopMBB, DL, TII->get(LoongArch::BEQ))
----------------
xen0n wrote:

`BEQZ` has the advantage over `BEQ` in that its reach is broader by 5 bits (width of a GPR slot), so if some of the changed bits potentially refers to a remote MBB, we may want to preserve them? I'm not looking at this code as closely as I'd prefer because I'm just battling my procrastination and doing a quick review here.

https://github.com/llvm/llvm-project/pull/139695


More information about the llvm-commits mailing list