[PATCH] D55744: [InstCombine] don't widen an arbitrary sequence of vector ops (PR40032)

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCasts.cpp:1117
   unsigned BitsToClear;
-  if ((DestTy->isVectorTy() || shouldChangeType(SrcTy, DestTy)) &&
+  if (!DestTy->isVectorTy() && shouldChangeType(SrcTy, DestTy) &&
       canEvaluateZExtd(Src, DestTy, BitsToClear, *this, &CI)) {
----------------
What about putting !DestTy->isVectorTy() inside InstCombiner::shouldChangeType ?


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

https://reviews.llvm.org/D55744