[PATCH] D111901: [VectorCombine] fold shuffle-of-binops with common operand

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:1084
+      TargetTransformInfo::SK_PermuteSingleSrc, VecTy, UnaryMask);
+  if (ShufCost > BinopCost)
+    return false;
----------------
Are we OK with accepting the fold if ShufCost == BinopCost ?


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

https://reviews.llvm.org/D111901