This is an archive of the discontinued LLVM Phabricator instance.

[Polly][ScopBuilder] Make -polly-stmt-granularity=scalar-indep the default.
ClosedPublic

Authored by Meinersbur on Jan 16 2018, 5:01 PM.

Details

Summary

Splitting basic blocks into multiple statements if there are now additional scalar dependencies gives more freedom to the scheduler, but more statements also means higher compile-time complexity. Switch to finer statement granularity, the additional compile time should be limited by the number of operations quota.

Diff Detail

Repository
rL LLVM

Event Timeline

This revision is now accepted and ready to land.Jan 16 2018, 11:10 PM
Meinersbur updated this revision to Diff 132726.Feb 2 2018, 9:49 PM

Instead of adding -polly-stmt-granulairy=bb globally, only add the flag to the tests that need it.

This avoids additional discrepancy between Polly's default settings and neew regression tests.

Many differences stem from statements for cmp instructions being created, but then simplified-away because such statments contain no write.

Waiting for confirmation this is ok before committing

This revision was automatically updated to reflect the committed changes.