[llvm] [WebAssembly] Fix missed optimization in 50142 (PR #144741)

Luke Lau via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 19 15:07:38 PDT 2025


================
@@ -3248,6 +3248,37 @@ static SDValue performSETCCCombine(SDNode *N,
   ISD::CondCode Cond = cast<CondCodeSDNode>(N->getOperand(2))->get();
   SDLoc DL(N);
   EVT VT = N->getValueType(0);
+  //  N           LHS     LhsL        LhsLL    LhsLR   InnerCond RHS Cond
+  // setcc (iN (bitcast (setcc vNi1 (vNiY X), <vNiY 0>, eq)),     0, eq
+  // => all_true (vNi1 X)
+  if (DCI.isBeforeLegalize() && VT.isScalarInteger() && (Cond == ISD::SETEQ) &&
----------------
lukel97 wrote:

Since @sparker-arm's patch is NFC and I don't think you need to change the existing combine, I don't think you need to stack/rebase your patch on top of it.
We don't really have a good system for stacking patches in LLVM anyway :)

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


More information about the llvm-commits mailing list