This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Merge TempScop into Scop
ClosedPublic

Authored by Meinersbur on Sep 11 2015, 7:46 AM.

Details

Summary

TempScop is basically a holder for AccFuncMap, the dictionary from BasicBlocks to IRAccess lists. We move the list into polly::Scop and remove the polly::TempScop class.

There is one small change in behavior: If ScopInfo finds that its AssumedContext is impossible, it bails out by deleting the Scop object. The TempScop::print (invoked with opt -polly-scops -analyze) cannot print the AccFuncMap anymore as it would with a separate TempScop.

Diff Detail

Event Timeline

Meinersbur updated this revision to Diff 34548.Sep 11 2015, 7:46 AM
Meinersbur retitled this revision from to [Polly] Merge TempScop into Scop.
Meinersbur updated this object.
Meinersbur added reviewers: grosser, jdoerfert.
Meinersbur added a project: Restricted Project.
Meinersbur added subscribers: llvm-commits, pollydev.
grosser accepted this revision.Sep 11 2015, 8:05 AM
grosser edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Sep 11 2015, 8:05 AM
Meinersbur updated this revision to Diff 34590.Sep 11 2015, 2:24 PM
Meinersbur edited edge metadata.

rebase to r247470

This revision was automatically updated to reflect the committed changes.