[PATCH] D74959: [VectorUtils] Move ToVectorTy to VectorUtils.h (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 08:50:46 PST 2020


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/InjectTLIMappings.cpp:64
-/// the scalar type.
-static Type *ToVectorTy(Type *Scalar, unsigned VF, bool isScalable = false) {
-  if (Scalar->isVoidTy() || VF == 1)
----------------
fpetrogalli wrote:
> Could we keep this one and remove the other one?
Ah right, I’ll keep the one with scalable


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74959





More information about the llvm-commits mailing list