[polly] r370368 - [ScopBuilder] Remove superfluous while loop in buildDomains. NFC.

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 07:42:41 PDT 2019


Author: meinersbur
Date: Thu Aug 29 07:42:41 2019
New Revision: 370368

URL: http://llvm.org/viewvc/llvm-project?rev=370368&view=rev
Log:
[ScopBuilder] Remove superfluous while loop in buildDomains. NFC.

The while loop iterating parent loop in ScopBuilder::buildDomains is
unnecessary because either L or LD are later unused, this is a simple
patch removing it.

Patch by bin.narwal <bin.narwal at gmail.com>

Differential Revision: https://reviews.llvm.org/D66698

Modified:
    polly/trunk/lib/Analysis/ScopBuilder.cpp

Modified: polly/trunk/lib/Analysis/ScopBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopBuilder.cpp?rev=370368&r1=370367&r2=370368&view=diff
==============================================================================
--- polly/trunk/lib/Analysis/ScopBuilder.cpp (original)
+++ polly/trunk/lib/Analysis/ScopBuilder.cpp Thu Aug 29 07:42:41 2019
@@ -831,10 +831,6 @@ bool ScopBuilder::buildDomains(
   auto *S =
       isl_set_universe(isl_space_set_alloc(scop->getIslCtx().get(), 0, LD + 1));
 
-  while (LD-- >= 0) {
-    L = L->getParentLoop();
-  }
-
   InvalidDomainMap[EntryBB] = isl::manage(isl_set_empty(isl_set_get_space(S)));
   isl::noexceptions::set Domain = isl::manage(S);
   scop->setDomain(EntryBB, Domain);




More information about the llvm-commits mailing list