[PATCH] D94708: [IR] Introduce llvm.experimental.vector.splice intrinsic

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:940
+  case ISD::VECTOR_SPLICE:
+    SplitVecRes_LOAD(cast<LoadSDNode>(TLI.expandVectorSplice(N, DAG)), Lo, Hi);
+    break;
----------------
This doesn't look great to me because it ties the hands of expandVectorSplice, whose name does not suggest it must return a load.  If you prefer not to explicitly create the split loads then it seems more natural, and future proof, if the result of expandVectorSplice is split using a pair of EXTRACT_SUBVECTOR operations.


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

https://reviews.llvm.org/D94708