[PATCH] D154738: [SLP]Introduce isValiVectorOp to check if the vector instruction is going to be scalarized.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:355
+
+  bool isLegalVectorIntrinsic(Intrinsic::ID Id, VectorType *VecTy) const {
+    unsigned ISD = ISD::DELETED_NODE;
----------------
getTypeBasedIntrinsicInstrCost already has a similar IntrinsicID->ISD conversion - merge them?


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:379
+    case Intrinsic::cos:
+      ISD = ISD::FSIN;
+      break;
----------------
ISD::FCOS?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154738