[PATCH] D107904: [LegalizeTypes][VP] Add widening support for binary VP ops

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 17 14:08:55 PDT 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:3428
+                     {InOp1, InOp2, Mask, N->getOperand(3)}, N->getFlags());
+}
+
----------------
Do we gain much by keeping this separate instead of just adding VP handling to WidenVecRes_Binary ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107904



More information about the llvm-commits mailing list