[PATCH] D72224: [LegalizeVectorOps] Improve handling of multi-result operations.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 6 06:46:44 PST 2020
arsenm added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp:145-148
+ std::pair<SDValue, SDValue> ExpandUADDSUBO(SDValue Op);
+ std::pair<SDValue, SDValue> ExpandSADDSUBO(SDValue Op);
+ std::pair<SDValue, SDValue> ExpandMULO(SDValue Op);
SDValue ExpandAddSubSat(SDValue Op);
----------------
Extra space here.
Is there a real point to returning SDValues here instead of just the SDNode*? I assume this always returns (N, 0), (N, 1)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72224/new/
https://reviews.llvm.org/D72224
More information about the llvm-commits
mailing list