This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by Meinersbur on Mar 15 2017, 9:38 AM.

Details

Summary

Introduce ScopStmt::getSurroundingLoop() to replace getFirstNonBoxedLoopFor.

getSurroundingLoop() returns the precomputed surrounding/first non-boxed loop. Except in ScopDetection, the list of boxed loops is only used to get the surrounding loop. getFirstNonBoxedLoopFor also requires LoopInfo at every use which is not necessarily available everywhere where we want to use it.

Diff Detail

Repository
rL LLVM

Event Timeline

Meinersbur created this revision.Mar 15 2017, 9:38 AM
grosser accepted this revision.Mar 15 2017, 11:33 AM

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

lib/Analysis/ScopInfo.cpp
2597 ↗(On Diff #91885)

Nice!

2714 ↗(On Diff #91885)

Nice!

This revision is now accepted and ready to land.Mar 15 2017, 11:33 AM
Meinersbur updated this revision to Diff 91942.Mar 15 2017, 3:25 PM
Meinersbur edited the summary of this revision. (Show Details)
  • No surrounding loop for copy stmts.
This revision was automatically updated to reflect the committed changes.