<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2016-02-02 15:14 GMT+01:00 Johannes Doerfert via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Author: jdoerfert<br>
Date: Tue Feb  2 08:14:40 2016<br>
New Revision: 259501<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=259501&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=259501&view=rev</a><br>
Log:<br>
Remove helper function [NFC]<br>
<br>
Modified:<br>
    polly/trunk/include/polly/ScopInfo.h<br>
    polly/trunk/lib/Analysis/ScopInfo.cpp<br>
<br>
Modified: polly/trunk/include/polly/ScopInfo.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/polly/trunk/include/polly/ScopInfo.h?rev=259501&r1=259500&r2=259501&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/polly/trunk/include/polly/ScopInfo.h?rev=259501&r1=259500&r2=259501&view=diff</a><br>
==============================================================================<br>
--- polly/trunk/include/polly/ScopInfo.h (original)<br>
+++ polly/trunk/include/polly/ScopInfo.h Tue Feb  2 08:14:40 2016<br>
@@ -1921,13 +1921,6 @@ class ScopInfo : public RegionPass {<br>
   Scop *scop;<br>
   isl_ctx *ctx;<br>
<br>
-  /// @brief Return the SCoP region that is currently processed.<br>
-  Region *getRegion() const {<br>
-    if (!scop)<br>
-      return nullptr;<br>
-    return &scop->getRegion();<br>
-  }<br>
-<br>
   // Clear the context.<br>
   void clear();<br>
<br>
@@ -1974,9 +1967,12 @@ class ScopInfo : public RegionPass {<br>
<br>
   /// @brief Create ScopStmt for all BBs and non-affine subregions of @p SR.<br>
   ///<br>
+  /// @param R  The SCoP region.<br>
+  /// @param SR A subregion of @p R.<br>
+  ///<br>
   /// Some of the statments might be optimized away later when they do not<br>
   /// access any memory and thus have no effect.<br>
-  void buildStmts(Region &SR);<br>
+  void buildStmts(Region &R, Region &SR);\\\<br>
<br>
   /// @brief Build the access functions for the basic block @p BB<br>
   ///<br>
<br>
Modified: polly/trunk/lib/Analysis/ScopInfo.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopInfo.cpp?rev=259501&r1=259500&r2=259501&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopInfo.cpp?rev=259501&r1=259500&r2=259501&view=diff</a><br>
==============================================================================<br>
--- polly/trunk/lib/Analysis/ScopInfo.cpp (original)<br>
+++ polly/trunk/lib/Analysis/ScopInfo.cpp Tue Feb  2 08:14:40 2016<br>
@@ -3876,17 +3876,16 @@ void ScopInfo::buildAccessFunctions(Regi<br>
       buildAccessFunctions(R, *I->getNodeAs<BasicBlock>());<br>
 }<br>
<br>
-void ScopInfo::buildStmts(Region &SR) {<br>
-  Region *R = getRegion();<br>
+void ScopInfo::buildStmts(Region &R, Region &SR) {<br></blockquote><div><br></div><div>Why? IMHO this makes it more confusing, as if buildStmts could operate on more than one Region. If it's for the sake of removing the helper function, you could directly use scop->getRegion() instead.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
-  if (SD->isNonAffineSubRegion(&SR, R)) {<br>
+  if (SD->isNonAffineSubRegion(&SR, &R)) {<br>
     scop->addScopStmt(nullptr, &SR);<br>
     return;<br>
   }<br>
<br>
   for (auto I = SR.element_begin(), E = SR.element_end(); I != E; ++I)<br>
     if (I->isSubRegion())<br>
-      buildStmts(*I->getNodeAs<Region>());<br>
+      buildStmts(R, *I->getNodeAs<Region>());<br>
     else<br>
       scop->addScopStmt(I->getNodeAs<BasicBlock>(), nullptr);<br>
 }<br>
@@ -4067,7 +4066,7 @@ void ScopInfo::buildScop(Region &R, Assu<br>
   unsigned MaxLoopDepth = getMaxLoopDepthInRegion(R, *LI, *SD);<br>
   scop = new Scop(R, AccFuncMap, *SD, *SE, *DT, *LI, ctx, MaxLoopDepth);<br>
<br>
-  buildStmts(R);<br>
+  buildStmts(R, R);<br>
   buildAccessFunctions(R, R);<br>
<br>
   // In case the region does not have an exiting block we will later (during<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>