[PATCH] D148523: [LegalizeTypes][VP] Widen fixed length vectors to VP nodes

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 17 06:16:59 PDT 2023


luke added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:4320
     // Operation doesn't trap so just widen as normal.
-    SDValue InOp1 = GetWidenedVector(N->getOperand(0));
-    SDValue InOp2 = GetWidenedVector(N->getOperand(1));
-    return DAG.getNode(N->getOpcode(), dl, WidenVT, InOp1, InOp2, Flags);
+    return WidenVecRes_Binary(N);
   }
----------------
This was changed so that it can reuse the `ShouldWidenToVP` logic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148523



More information about the llvm-commits mailing list