Reuse ScopStmt::isEmpty() function
Introduce a function getStmtForRegionNode() to the corresponding
ScopStmt of a RegionNode. We can use it to call the existing
ScopStmt::isEmpty() function instead of searching for accesses.
llvm-svn: 255465
diff --git a/polly/lib/Analysis/ScopInfo.cpp b/polly/lib/Analysis/ScopInfo.cpp
index 7e236f6..9d7a433 100644
--- a/polly/lib/Analysis/ScopInfo.cpp
+++ b/polly/lib/Analysis/ScopInfo.cpp
@@ -3318,15 +3318,14 @@
bool Scop::isIgnored(RegionNode *RN) {
BasicBlock *BB = getRegionNodeBasicBlock(RN);
+ ScopStmt *Stmt = getStmtForRegionNode(RN);
+
+ // If there is no stmt, then it already has been removed.
+ if (!Stmt)
+ return true;
// Check if there are accesses contained.
- bool ContainsAccesses = false;
- if (!RN->isSubRegion())
- ContainsAccesses = getAccessFunctions(BB);
- else
- for (BasicBlock *RBB : RN->getNodeAs<Region>()->blocks())
- ContainsAccesses |= (getAccessFunctions(RBB) != nullptr);
- if (!ContainsAccesses)
+ if (Stmt->isEmpty())
return true;
// Check for reachability via non-error blocks.
@@ -3491,6 +3490,10 @@
return StmtMapIt->second;
}
+ScopStmt *Scop::getStmtForRegionNode(RegionNode *RN) const {
+ return getStmtForBasicBlock(getRegionNodeBasicBlock(RN));
+}
+
int Scop::getRelativeLoopDepth(const Loop *L) const {
Loop *OuterLoop =
L ? R.outermostLoopInRegion(const_cast<Loop *>(L)) : nullptr;