[PATCH] D36571: [DAGCombiner] combine vextract (v1iX extract_subvector(vNiX, Idx)) into vextract(vNiX, Idx).

Elad Cohen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 08:10:32 PDT 2017


eladcohen added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:13790
+                       InVec->getOperand(0), InVec->getOperand(1));
+  }
+
----------------
RKSimon wrote:
> Would this be better in SelectionDAG::getNode() ?
Sounds good. I'll move it.


https://reviews.llvm.org/D36571





More information about the llvm-commits mailing list