[PATCH] D30985: [Polly][ScopInfo] Introduce ScopStmt::getSurroundingLoop(). NFC.

Tobias Grosser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 11:33:17 PDT 2017


grosser accepted this revision.
grosser added a comment.
This revision is now accepted and ready to land.

LGTM, if you drop the surrounding loop stuff from copystmt.



================
Comment at: lib/Analysis/ScopInfo.cpp:2597
+  auto *ExitStmt = getStmtFor(ExitBB);
+  auto *ExitBBLoop = ExitStmt->getSurroundingLoop();
 
----------------
Nice!


================
Comment at: lib/Analysis/ScopInfo.cpp:2714
 
-      auto &BoxedLoops = getBoxedLoops();
-      auto *SuccBBLoop = getFirstNonBoxedLoopFor(SuccBB, LI, BoxedLoops);
+      auto *SuccBBLoop = SuccStmt->getSurroundingLoop();
       CondSet = adjustDomainDimensions(*this, CondSet, BBLoop, SuccBBLoop);
----------------
Nice!


https://reviews.llvm.org/D30985





More information about the llvm-commits mailing list