Skip to content

Commit ee468ec

Browse files
[LoopVectorize][AArch64][SVE] Generate wide active lane masks
This patch makes the LoopVectorize generate lane masks longer than the VF to allow the target to better utilise the instruction set. The vectorizer emit one or more wide `llvm.get.active.lane.mask.*` calls plus several `llvm.vector.extract.*` calls to yield the required number of VF-wide masks. The motivating exammple is a vectorised loop with unroll factor 2 that can use the SVE2.1 `whilelo` instruction with predicate pair result, or a SVE `whilelo` instruction with smaller element size plus `punpklo`/`punpkhi`. How wide is the lane mask that the vectoriser emits is controlled by a TargetTransformInfo hook `getMaxPredicateLength`.The default impementation (return the same length as the VF) keeps the change non-functional for targets that can't or are not prepared to handle wider lane masks.
1 parent ae86278 commit ee468ec

23 files changed

+3235
-1336
lines changed

llvm/include/llvm/Analysis/TargetTransformInfo.h

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1244,6 +1244,8 @@ class TargetTransformInfo {
12441244
/// and the number of execution units in the CPU.
12451245
unsigned getMaxInterleaveFactor(ElementCount VF) const;
12461246

1247+
ElementCount getMaxPredicateLength(ElementCount VF) const;
1248+
12471249
/// Collect properties of V used in cost analysis, e.g. OP_PowerOf2.
12481250
static OperandValueInfo getOperandInfo(const Value *V);
12491251

@@ -2002,6 +2004,9 @@ class TargetTransformInfo::Concept {
20022004
virtual bool shouldPrefetchAddressSpace(unsigned AS) const = 0;
20032005

20042006
virtual unsigned getMaxInterleaveFactor(ElementCount VF) = 0;
2007+
2008+
virtual ElementCount getMaxPredicateLength(ElementCount VF) const = 0;
2009+
20052010
virtual InstructionCost getArithmeticInstrCost(
20062011
unsigned Opcode, Type *Ty, TTI::TargetCostKind CostKind,
20072012
OperandValueInfo Opd1Info, OperandValueInfo Opd2Info,
@@ -2627,6 +2632,11 @@ class TargetTransformInfo::Model final : public TargetTransformInfo::Concept {
26272632
unsigned getMaxInterleaveFactor(ElementCount VF) override {
26282633
return Impl.getMaxInterleaveFactor(VF);
26292634
}
2635+
2636+
ElementCount getMaxPredicateLength(ElementCount VF) const override {
2637+
return Impl.getMaxPredicateLength(VF);
2638+
}
2639+
26302640
unsigned getEstimatedNumberOfCaseClusters(const SwitchInst &SI,
26312641
unsigned &JTSize,
26322642
ProfileSummaryInfo *PSI,

llvm/include/llvm/Analysis/TargetTransformInfoImpl.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -537,6 +537,8 @@ class TargetTransformInfoImplBase {
537537

538538
unsigned getMaxInterleaveFactor(ElementCount VF) const { return 1; }
539539

540+
ElementCount getMaxPredicateLength(ElementCount VF) const { return VF; }
541+
540542
InstructionCost getArithmeticInstrCost(
541543
unsigned Opcode, Type *Ty, TTI::TargetCostKind CostKind,
542544
TTI::OperandValueInfo Opd1Info, TTI::OperandValueInfo Opd2Info,

llvm/include/llvm/CodeGen/BasicTTIImpl.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -890,6 +890,8 @@ class BasicTTIImplBase : public TargetTransformInfoImplCRTPBase<T> {
890890

891891
unsigned getMaxInterleaveFactor(ElementCount VF) { return 1; }
892892

893+
ElementCount getMaxPredicateLength(ElementCount VF) const { return VF; }
894+
893895
InstructionCost getArithmeticInstrCost(
894896
unsigned Opcode, Type *Ty, TTI::TargetCostKind CostKind,
895897
TTI::OperandValueInfo Opd1Info = {TTI::OK_AnyValue, TTI::OP_None},

llvm/lib/Analysis/TargetTransformInfo.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -821,6 +821,10 @@ unsigned TargetTransformInfo::getMaxInterleaveFactor(ElementCount VF) const {
821821
return TTIImpl->getMaxInterleaveFactor(VF);
822822
}
823823

824+
ElementCount TargetTransformInfo::getMaxPredicateLength(ElementCount VF) const {
825+
return TTIImpl->getMaxPredicateLength(VF);
826+
}
827+
824828
TargetTransformInfo::OperandValueInfo
825829
TargetTransformInfo::getOperandInfo(const Value *V) {
826830
OperandValueKind OpInfo = OK_AnyValue;

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3383,6 +3383,15 @@ unsigned AArch64TTIImpl::getMaxInterleaveFactor(ElementCount VF) {
33833383
return ST->getMaxInterleaveFactor();
33843384
}
33853385

3386+
ElementCount AArch64TTIImpl::getMaxPredicateLength(ElementCount VF) const {
3387+
// Do not create masks bigger than `<vscale x 16 x i1>`.
3388+
unsigned N = ST->hasSVE() ? 16 : 0;
3389+
// Do not create masks that are more than twice the VF.
3390+
N = std::min(N, 2 * VF.getKnownMinValue());
3391+
return VF.isScalable() ? ElementCount::getScalable(N)
3392+
: ElementCount::getFixed(N);
3393+
}
3394+
33863395
// For Falkor, we want to avoid having too many strided loads in a loop since
33873396
// that can exhaust the HW prefetcher resources. We adjust the unroller
33883397
// MaxCount preference below to attempt to ensure unrolling doesn't create too

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,8 @@ class AArch64TTIImpl : public BasicTTIImplBase<AArch64TTIImpl> {
157157

158158
unsigned getMaxInterleaveFactor(ElementCount VF);
159159

160+
ElementCount getMaxPredicateLength(ElementCount VF) const;
161+
160162
bool prefersVectorizedAddressing() const;
161163

162164
InstructionCost getMaskedMemoryOpCost(unsigned Opcode, Type *Src,

llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,14 @@ class VPBuilder {
202202
VPValue *createICmp(CmpInst::Predicate Pred, VPValue *A, VPValue *B,
203203
DebugLoc DL = {}, const Twine &Name = "");
204204

205+
VPValue *createGetActiveLaneMask(VPValue *IV, VPValue *TC, DebugLoc DL,
206+
const Twine &Name = "") {
207+
auto *ALM = new VPActiveLaneMaskRecipe(IV, TC, DL, Name);
208+
if (BB)
209+
BB->insert(ALM, InsertPt);
210+
return ALM;
211+
}
212+
205213
//===--------------------------------------------------------------------===//
206214
// RAII helpers.
207215
//===--------------------------------------------------------------------===//

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -588,6 +588,10 @@ class InnerLoopVectorizer {
588588
/// count of the original loop for both main loop and epilogue vectorization.
589589
void setTripCount(Value *TC) { TripCount = TC; }
590590

591+
ElementCount getMaxPredicateLength(ElementCount VF) const {
592+
return TTI->getMaxPredicateLength(VF);
593+
}
594+
591595
protected:
592596
friend class LoopVectorizationPlanner;
593597

@@ -7509,7 +7513,8 @@ LoopVectorizationPlanner::executePlan(
75097513
LLVM_DEBUG(BestVPlan.dump());
75107514

75117515
// Perform the actual loop transformation.
7512-
VPTransformState State(BestVF, BestUF, LI, DT, ILV.Builder, &ILV, &BestVPlan,
7516+
VPTransformState State(BestVF, BestUF, TTI.getMaxPredicateLength(BestVF), LI,
7517+
DT, ILV.Builder, &ILV, &BestVPlan,
75137518
OrigLoop->getHeader()->getContext());
75147519

75157520
// 0. Generate SCEV-dependent code into the preheader, including TripCount,

llvm/lib/Transforms/Vectorize/VPlan.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -215,12 +215,13 @@ VPBasicBlock::iterator VPBasicBlock::getFirstNonPhi() {
215215
return It;
216216
}
217217

218-
VPTransformState::VPTransformState(ElementCount VF, unsigned UF, LoopInfo *LI,
218+
VPTransformState::VPTransformState(ElementCount VF, unsigned UF,
219+
ElementCount MaxPred, LoopInfo *LI,
219220
DominatorTree *DT, IRBuilderBase &Builder,
220221
InnerLoopVectorizer *ILV, VPlan *Plan,
221222
LLVMContext &Ctx)
222-
: VF(VF), UF(UF), CFG(DT), LI(LI), Builder(Builder), ILV(ILV), Plan(Plan),
223-
LVer(nullptr),
223+
: VF(VF), UF(UF), MaxPred(MaxPred), CFG(DT), LI(LI), Builder(Builder),
224+
ILV(ILV), Plan(Plan), LVer(nullptr),
224225
TypeAnalysis(Plan->getCanonicalIV()->getScalarType(), Ctx) {}
225226

226227
Value *VPTransformState::get(VPValue *Def, const VPIteration &Instance) {

llvm/lib/Transforms/Vectorize/VPlan.h

Lines changed: 47 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -235,13 +235,14 @@ struct VPIteration {
235235
/// VPTransformState holds information passed down when "executing" a VPlan,
236236
/// needed for generating the output IR.
237237
struct VPTransformState {
238-
VPTransformState(ElementCount VF, unsigned UF, LoopInfo *LI,
239-
DominatorTree *DT, IRBuilderBase &Builder,
238+
VPTransformState(ElementCount VF, unsigned UF, ElementCount MaxPred,
239+
LoopInfo *LI, DominatorTree *DT, IRBuilderBase &Builder,
240240
InnerLoopVectorizer *ILV, VPlan *Plan, LLVMContext &Ctx);
241241

242242
/// The chosen Vectorization and Unroll Factors of the loop being vectorized.
243243
ElementCount VF;
244244
unsigned UF;
245+
ElementCount MaxPred;
245246

246247
/// Hold the indices to generate specific scalar instructions. Null indicates
247248
/// that all instances are to be generated, using either scalar or vector
@@ -1174,7 +1175,6 @@ class VPInstruction : public VPRecipeWithIRFlags {
11741175
Not,
11751176
SLPLoad,
11761177
SLPStore,
1177-
ActiveLaneMask,
11781178
ExplicitVectorLength,
11791179
CalculateTripCountMinusVF,
11801180
// Increment the canonical IV separately for each unrolled part.
@@ -1329,6 +1329,50 @@ class VPInstruction : public VPRecipeWithIRFlags {
13291329
}
13301330
};
13311331

1332+
class VPActiveLaneMaskRecipe : public VPRecipeWithIRFlags {
1333+
const std::string Name;
1334+
1335+
public:
1336+
VPActiveLaneMaskRecipe(VPValue *IV, VPValue *TC, DebugLoc DL = {},
1337+
const Twine &Name = "")
1338+
: VPRecipeWithIRFlags(VPDef::VPActiveLaneMaskSC,
1339+
std::initializer_list<VPValue *>{IV, TC}, DL),
1340+
Name(Name.str()) {}
1341+
1342+
VP_CLASSOF_IMPL(VPDef::VPActiveLaneMaskSC)
1343+
1344+
VPActiveLaneMaskRecipe *clone() override {
1345+
SmallVector<VPValue *, 2> Operands(operands());
1346+
assert(Operands.size() == 2 && "by construction");
1347+
auto *New = new VPActiveLaneMaskRecipe(Operands[0], Operands[1],
1348+
getDebugLoc(), Name);
1349+
New->transferFlags(*this);
1350+
return New;
1351+
}
1352+
1353+
void execute(VPTransformState &State) override;
1354+
1355+
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
1356+
/// Print the recipe.
1357+
void print(raw_ostream &O, const Twine &Indent,
1358+
VPSlotTracker &SlotTracker) const override;
1359+
#endif
1360+
1361+
bool onlyFirstLaneUsed(const VPValue *Op) const override {
1362+
assert(is_contained(operands(), Op) &&
1363+
"Op must be an operand of the recipe");
1364+
1365+
return true;
1366+
}
1367+
1368+
bool onlyFirstPartUsed(const VPValue *Op) const override {
1369+
assert(is_contained(operands(), Op) &&
1370+
"Op must be an operand of the recipe");
1371+
1372+
return false;
1373+
}
1374+
};
1375+
13321376
/// VPWidenRecipe is a recipe for producing a copy of vector type its
13331377
/// ingredient. This recipe covers most of the traditional vectorization cases
13341378
/// where each ingredient transforms into a vectorized version of itself.

0 commit comments

Comments
 (0)