[PATCH] D78357: [NFC][TTI] Explicit use of VectorType

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 05:22:13 PDT 2020


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - you just need to clean up a few missed uses of auto with cast<>. Cheers.



================
Comment at: llvm/lib/Analysis/TargetTransformInfo.cpp:1305
+    VectorType *Ty = cast<VectorType>(Shuffle->getType());
+    VectorType *SrcTy = cast<VectorType>(Shuffle->getOperand(0)->getType());
 
----------------
auto


================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:61
   Type *ScalarTy = Ext0->getType();
-  Type *VecTy = Ext0->getOperand(0)->getType();
+  VectorType *VecTy = cast<VectorType>(Ext0->getOperand(0)->getType());
   int ScalarOpCost, VectorOpCost;
----------------
auto


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

https://reviews.llvm.org/D78357





More information about the llvm-commits mailing list