diff --git a/include/llvm/Analysis/TargetTransformInfo.h b/include/llvm/Analysis/TargetTransformInfo.h index 992e15500d9..b2eb9864fe0 100644 --- a/include/llvm/Analysis/TargetTransformInfo.h +++ b/include/llvm/Analysis/TargetTransformInfo.h @@ -945,7 +945,8 @@ public: /// \return The minimum vectorization factor for types of given element /// bit width, or 0 if there is no minimum VF. The returned value only /// applies when shouldMaximizeVectorBandwidth returns true. - unsigned getMinimumVF(unsigned ElemWidth) const; + /// If IsScalable is true, the returned ElementCount must be a scalable VF. + ElementCount getMinimumVF(unsigned ElemWidth, bool IsScalable) const; /// \return The maximum vectorization factor for types of given element /// bit width and opcode, or 0 if there is no maximum VF. @@ -1523,7 +1524,8 @@ public: virtual unsigned getMinVectorRegisterBitWidth() = 0; virtual Optional getMaxVScale() const = 0; virtual bool shouldMaximizeVectorBandwidth(bool OptSize) const = 0; - virtual unsigned getMinimumVF(unsigned ElemWidth) const = 0; + virtual ElementCount getMinimumVF(unsigned ElemWidth, + bool IsScalable) const = 0; virtual unsigned getMaximumVF(unsigned ElemWidth, unsigned Opcode) const = 0; virtual bool shouldConsiderAddressTypePromotion( const Instruction &I, bool &AllowPromotionWithoutCommonHeader) = 0; @@ -1951,8 +1953,9 @@ public: bool shouldMaximizeVectorBandwidth(bool OptSize) const override { return Impl.shouldMaximizeVectorBandwidth(OptSize); } - unsigned getMinimumVF(unsigned ElemWidth) const override { - return Impl.getMinimumVF(ElemWidth); + ElementCount getMinimumVF(unsigned ElemWidth, + bool IsScalable) const override { + return Impl.getMinimumVF(ElemWidth, IsScalable); } unsigned getMaximumVF(unsigned ElemWidth, unsigned Opcode) const override { return Impl.getMaximumVF(ElemWidth, Opcode); diff --git a/include/llvm/Analysis/TargetTransformInfoImpl.h b/include/llvm/Analysis/TargetTransformInfoImpl.h index a4f9e1d7b98..f0443fb1d73 100644 --- a/include/llvm/Analysis/TargetTransformInfoImpl.h +++ b/include/llvm/Analysis/TargetTransformInfoImpl.h @@ -374,7 +374,9 @@ public: bool shouldMaximizeVectorBandwidth(bool OptSize) const { return false; } - unsigned getMinimumVF(unsigned ElemWidth) const { return 0; } + ElementCount getMinimumVF(unsigned ElemWidth, bool IsScalable) const { + return ElementCount::get(0, IsScalable); + } unsigned getMaximumVF(unsigned ElemWidth, unsigned Opcode) const { return 0; } diff --git a/lib/Analysis/TargetTransformInfo.cpp b/lib/Analysis/TargetTransformInfo.cpp index e376d42c15b..ee1573098ed 100644 --- a/lib/Analysis/TargetTransformInfo.cpp +++ b/lib/Analysis/TargetTransformInfo.cpp @@ -640,8 +640,9 @@ bool TargetTransformInfo::shouldMaximizeVectorBandwidth(bool OptSize) const { return TTIImpl->shouldMaximizeVectorBandwidth(OptSize); } -unsigned TargetTransformInfo::getMinimumVF(unsigned ElemWidth) const { - return TTIImpl->getMinimumVF(ElemWidth); +ElementCount TargetTransformInfo::getMinimumVF(unsigned ElemWidth, + bool IsScalable) const { + return TTIImpl->getMinimumVF(ElemWidth, IsScalable); } unsigned TargetTransformInfo::getMaximumVF(unsigned ElemWidth, diff --git a/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp b/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp index 1cefa6a0464..af7bc468224 100644 --- a/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp +++ b/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp @@ -104,8 +104,10 @@ unsigned HexagonTTIImpl::getMinVectorRegisterBitWidth() const { return useHVX() ? ST.getVectorLength()*8 : 32; } -unsigned HexagonTTIImpl::getMinimumVF(unsigned ElemWidth) const { - return (8 * ST.getVectorLength()) / ElemWidth; +ElementCount HexagonTTIImpl::getMinimumVF(unsigned ElemWidth, + bool IsScalable) const { + assert(!IsScalable && "Scalable VFs are not supported for Hexagon"); + return ElementCount::getFixed((8 * ST.getVectorLength()) / ElemWidth); } unsigned HexagonTTIImpl::getScalarizationOverhead(VectorType *Ty, diff --git a/lib/Target/Hexagon/HexagonTargetTransformInfo.h b/lib/Target/Hexagon/HexagonTargetTransformInfo.h index 835358d3fed..dc075d6147b 100644 --- a/lib/Target/Hexagon/HexagonTargetTransformInfo.h +++ b/lib/Target/Hexagon/HexagonTargetTransformInfo.h @@ -82,7 +82,7 @@ public: unsigned getMaxInterleaveFactor(unsigned VF); unsigned getRegisterBitWidth(bool Vector) const; unsigned getMinVectorRegisterBitWidth() const; - unsigned getMinimumVF(unsigned ElemWidth) const; + ElementCount getMinimumVF(unsigned ElemWidth, bool IsScalable) const; bool shouldMaximizeVectorBandwidth(bool OptSize) const { return true; diff --git a/lib/Transforms/Vectorize/LoopVectorize.cpp b/lib/Transforms/Vectorize/LoopVectorize.cpp index 71b626e8683..cb7bc82fb04 100644 --- a/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -5812,7 +5812,8 @@ LoopVectorizationCostModel::computeFeasibleMaxVF(unsigned ConstTripCount, break; } } - if (auto MinVF = ElementCount::getFixed(TTI.getMinimumVF(SmallestType))) { + if (ElementCount MinVF = + TTI.getMinimumVF(SmallestType, /*IsScalable=*/false)) { if (ElementCount::isKnownLT(MaxVF, MinVF)) { LLVM_DEBUG(dbgs() << "LV: Overriding calculated MaxVF(" << MaxVF << ") with target's minimum: " << MinVF << '\n');