This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Create Scop name lazily
ClosedPublic

Authored by philip.pfaffe on May 15 2018, 3:52 AM.

Details

Summary

Creating the Scop name is expensive, because creating the Region name
it's derived from is expensive. So create the name lazily, because getName() is
actually called rarely.

Side note: getName() and getNameStr() return different things. Is this
intentional? Should it be unified?

Event Timeline

philip.pfaffe created this revision.May 15 2018, 3:52 AM
grosser accepted this revision.May 15 2018, 5:18 AM

LGTM. Can you potentially include the old SVN revisions in your commit message?

This revision is now accepted and ready to land.May 15 2018, 5:18 AM
This revision was automatically updated to reflect the committed changes.