[llvm-commits] [polly] r131356 - in /polly/trunk: include/polly/ScopDetection.h lib/Analysis/ScopDetection.cpp lib/CodeGeneration.cpp

[Tobias Grosser]

Author: grosser
Date: Sat May 14 14:02:21 2011
New Revision: 131356

URL: http://llvm.org/viewvc/llvm-project?rev=131356&view=rev
Log:
CodeGeneration: No need to forget SCoP.

As we do not delete the SCoP, there is no need to remove it from the SCoP
detection.

Modified:
    polly/trunk/include/polly/ScopDetection.h
    polly/trunk/lib/Analysis/ScopDetection.cpp
    polly/trunk/lib/CodeGeneration.cpp

Modified: polly/trunk/include/polly/ScopDetection.h
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/include/polly/ScopDetection.h?rev=131356&r1=131355&r2=131356&view=diff
==============================================================================
--- polly/trunk/include/polly/ScopDetection.h (original)
+++ polly/trunk/include/polly/ScopDetection.h Sat May 14 14:02:21 2011
@@ -244,11 +244,6 @@
   const_iterator end()   const { return ValidRegions.end();   }
   //@}
 
-  /// @brief Remove a region and its children from valid region set.
-  ///
-  /// @param R The region to remove.
-  void forgetScop(const Region &R);
-
   /// @brief Mark the function as invalid so we will not extract any scop from
   ///        the function.
   ///

Modified: polly/trunk/lib/Analysis/ScopDetection.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopDetection.cpp?rev=131356&r1=131355&r2=131356&view=diff
==============================================================================
--- polly/trunk/lib/Analysis/ScopDetection.cpp (original)
+++ polly/trunk/lib/Analysis/ScopDetection.cpp Sat May 14 14:02:21 2011
@@ -624,11 +624,6 @@
     verifyRegion(**I);
 }
 
-void ScopDetection::forgetScop(const Region &R) {
-  assert(isMaxRegionInScop(R) && "R is not a Scop!");
-  ValidRegions.erase(&R);
-}
-
 void ScopDetection::getAnalysisUsage(AnalysisUsage &AU) const {
   AU.addRequired<DominatorTree>();
   AU.addRequired<PostDominatorTree>();

Modified: polly/trunk/lib/CodeGeneration.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/CodeGeneration.cpp?rev=131356&r1=131355&r2=131356&view=diff
==============================================================================
--- polly/trunk/lib/CodeGeneration.cpp (original)
+++ polly/trunk/lib/CodeGeneration.cpp Sat May 14 14:02:21 2011
@@ -1470,9 +1470,6 @@
 
     mergeControlFlow(splitBlock, &builder);
 
-    // Forget the Scop.
-    SD->forgetScop(*region);
-
     return false;
   }