[llvm] [RISCV] Handle scalable ops with < EEW / 2 narrow types in combineBinOp_VLToVWBinOp_VL (PR #84158)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 16:32:43 PDT 2024


================
@@ -13644,24 +13644,17 @@ struct NodeExtensionHelper {
     SupportsFPExt = false;
     EnforceOneUse = true;
     CheckMask = true;
+    SDValue NarrowOp;
     unsigned Opc = OrigOperand.getOpcode();
     switch (Opc) {
     case ISD::ZERO_EXTEND:
     case ISD::SIGN_EXTEND: {
+      NarrowOp = OrigOperand.getOperand(0);
----------------
topperc wrote:

Can't se just check for i1 in this case? None of the other cases can have an i1 narrow op past type legalization.

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


More information about the llvm-commits mailing list