[PATCH] D80235: [SVE] Eliminate calls to VectorType::getNumElements from BasicTTIImpl.h

[Sam Parker via Phabricator via llvm-commits]

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

There's quite a few casts of ShuffleVectorInst types and I was wondering whether this class will also be ported to the Fixed type?



================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:1128
+          ScalarizationCost += getScalarizationOverhead(
+              cast<FixedVectorType>(RetTy), true, false);
         ScalarizationCost += getOperandsScalarizationOverhead(Args, VF);
----------------
I'm assuming this cast isn't necessary?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80235