[polly] r265284 - Factor out "adjustDomainDimensions" function [NFC]
Johannes Doerfert via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 4 00:50:40 PDT 2016
Author: jdoerfert
Date: Mon Apr 4 02:50:40 2016
New Revision: 265284
URL: http://llvm.org/viewvc/llvm-project?rev=265284&view=rev
Log:
Factor out "adjustDomainDimensions" function [NFC]
Modified:
polly/trunk/lib/Analysis/ScopInfo.cpp
Modified: polly/trunk/lib/Analysis/ScopInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopInfo.cpp?rev=265284&r1=265283&r2=265284&view=diff
==============================================================================
--- polly/trunk/lib/Analysis/ScopInfo.cpp (original)
+++ polly/trunk/lib/Analysis/ScopInfo.cpp Mon Apr 4 02:50:40 2016
@@ -2188,6 +2188,55 @@ getFirstNonBoxedLoopFor(BasicBlock *BB,
return L;
}
+/// @brief Adjust the dimensions of @p Dom that was constructed for @p OldL
+/// to be compatible to domains constructed for loop @p NewL.
+///
+/// This function assumes @p NewL and @p OldL are equal or there is a CFG
+/// edge from @p OldL to @p NewL.
+static __isl_give isl_set *adjustDomainDimensions(Scop &S,
+ __isl_take isl_set *Dom,
+ Loop *OldL, Loop *NewL) {
+
+ // If the loops are the same there is nothing to do.
+ if (NewL == OldL)
+ return Dom;
+
+ int OldDepth = S.getRelativeLoopDepth(OldL);
+ int NewDepth = S.getRelativeLoopDepth(NewL);
+ // If both loops are non-affine loops there is nothing to do.
+ if (OldDepth == -1 && NewDepth == -1)
+ return Dom;
+
+ // Distinguish three cases:
+ // 1) The depth is the same but the loops are not.
+ // => One loop was left one was entered.
+ // 2) The depth increased from OldL to NewL.
+ // => One loop was entered, none was left.
+ // 3) The depth decreased from OldL to NewL.
+ // => Loops were left were difference of the depths defines how many.
+ if (OldDepth == NewDepth) {
+ assert(OldL->getParentLoop() == NewL->getParentLoop());
+ Dom = isl_set_project_out(Dom, isl_dim_set, NewDepth, 1);
+ Dom = isl_set_add_dims(Dom, isl_dim_set, 1);
+ Dom = addDomainDimId(Dom, NewDepth, NewL);
+ } else if (OldDepth < NewDepth) {
+ assert(OldDepth + 1 == NewDepth);
+ auto &R = S.getRegion();
+ (void)R;
+ assert(NewL->getParentLoop() == OldL ||
+ ((!OldL || !R.contains(OldL)) && R.contains(NewL)));
+ Dom = isl_set_add_dims(Dom, isl_dim_set, 1);
+ Dom = addDomainDimId(Dom, NewDepth, NewL);
+ } else {
+ assert(OldDepth > NewDepth);
+ int Diff = OldDepth - NewDepth;
+ int NumDim = isl_set_n_dim(Dom);
+ assert(NumDim >= Diff);
+ Dom = isl_set_project_out(Dom, isl_dim_set, NumDim - Diff, Diff);
+ }
+
+ return Dom;
+}
bool Scop::buildDomainsWithBranchConstraints(Region *R, ScopDetection &SD,
DominatorTree &DT, LoopInfo &LI) {
auto &BoxedLoops = *SD.getBoxedLoops(&getRegion());
@@ -2231,7 +2280,6 @@ bool Scop::buildDomainsWithBranchConstra
continue;
Loop *BBLoop = getRegionNodeLoop(RN, LI);
- int BBLoopDepth = getRelativeLoopDepth(BBLoop);
// Build the condition sets for the successor nodes of the current region
// node. If it is a non-affine subregion we will always execute the single
@@ -2258,32 +2306,8 @@ bool Scop::buildDomainsWithBranchConstra
continue;
}
- // Do not adjust the number of dimensions if we enter a boxed loop or are
- // in a non-affine subregion or if the surrounding loop stays the same.
auto *SuccBBLoop = getFirstNonBoxedLoopFor(SuccBB, LI, BoxedLoops);
-
- if (BBLoop != SuccBBLoop) {
-
- // Check if the edge to SuccBB is a loop entry or exit edge. If so
- // adjust the dimensionality accordingly. Lastly, if we leave a loop
- // and enter a new one we need to drop the old constraints.
- int SuccBBLoopDepth = getRelativeLoopDepth(SuccBBLoop);
- unsigned LoopDepthDiff = std::abs(BBLoopDepth - SuccBBLoopDepth);
- if (BBLoopDepth > SuccBBLoopDepth) {
- CondSet = isl_set_project_out(CondSet, isl_dim_set,
- isl_set_n_dim(CondSet) - LoopDepthDiff,
- LoopDepthDiff);
- } else if (SuccBBLoopDepth > BBLoopDepth) {
- assert(LoopDepthDiff == 1);
- CondSet = isl_set_add_dims(CondSet, isl_dim_set, 1);
- CondSet = addDomainDimId(CondSet, SuccBBLoopDepth, SuccBBLoop);
- } else if (BBLoopDepth >= 0) {
- assert(LoopDepthDiff <= 1);
- CondSet = isl_set_project_out(CondSet, isl_dim_set, BBLoopDepth, 1);
- CondSet = isl_set_add_dims(CondSet, isl_dim_set, 1);
- CondSet = addDomainDimId(CondSet, SuccBBLoopDepth, SuccBBLoop);
- }
- }
+ CondSet = adjustDomainDimensions(*this, CondSet, BBLoop, SuccBBLoop);
// Set the domain for the successor or merge it with an existing domain in
// case there are multiple paths (without loop back edges) to the
@@ -2370,7 +2394,6 @@ void Scop::propagateDomainConstraints(Re
}
Loop *BBLoop = getRegionNodeLoop(RN, LI);
- int BBLoopDepth = getRelativeLoopDepth(BBLoop);
isl_set *PredDom = isl_set_empty(isl_set_get_space(Domain));
for (auto *PredBB : predecessors(BB)) {
@@ -2386,32 +2409,10 @@ void Scop::propagateDomainConstraints(Re
PredBBDom = isl_set_universe(isl_set_get_space(PredDom));
if (!PredBBDom) {
- // Determine the loop depth of the predecessor and adjust its domain to
- // the domain of the current block. This can mean we have to:
- // o) Drop a dimension if this block is the exit of a loop, not the
- // header of a new loop and the predecessor was part of the loop.
- // o) Add an unconstrainted new dimension if this block is the header
- // of a loop and the predecessor is not part of it.
- // o) Drop the information about the innermost loop dimension when the
- // predecessor and the current block are surrounded by different
- // loops in the same depth.
PredBBDom = getDomainForBlock(PredBB, DomainMap, *R->getRegionInfo());
auto *PredBBLoop = getFirstNonBoxedLoopFor(PredBB, LI, BoxedLoops);
-
- int PredBBLoopDepth = getRelativeLoopDepth(PredBBLoop);
- unsigned LoopDepthDiff = std::abs(BBLoopDepth - PredBBLoopDepth);
- if (BBLoopDepth < PredBBLoopDepth)
- PredBBDom = isl_set_project_out(
- PredBBDom, isl_dim_set, isl_set_n_dim(PredBBDom) - LoopDepthDiff,
- LoopDepthDiff);
- else if (PredBBLoopDepth < BBLoopDepth) {
- assert(LoopDepthDiff == 1);
- PredBBDom = isl_set_add_dims(PredBBDom, isl_dim_set, 1);
- } else if (BBLoop != PredBBLoop && BBLoopDepth >= 0) {
- assert(LoopDepthDiff <= 1);
- PredBBDom = isl_set_drop_constraints_involving_dims(
- PredBBDom, isl_dim_set, BBLoopDepth, 1);
- }
+ PredBBDom =
+ adjustDomainDimensions(*this, PredBBDom, PredBBLoop, BBLoop);
}
PredDom = isl_set_union(PredDom, PredBBDom);
More information about the llvm-commits
mailing list