This is an archive of the discontinued LLVM Phabricator instance.

[SCEV] Re-apply r335197 (with Polly fixes).
ClosedPublic

Authored by timshen on Jun 21 2018, 1:49 PM.

Details

Summary

This initiates a discussion on changing Polly accordingly while re-applying r335197 (D48338).

I have never worked on Polly. The proposed change to param_div_div_div_2.ll is not educated, but just patterns that match the output.

All LLVM files are already reviewed in D48338.

Diff Detail

Repository
rL LLVM

Event Timeline

timshen created this revision.Jun 21 2018, 1:49 PM
efriedma accepted this revision.Jun 21 2018, 2:15 PM
efriedma added a subscriber: efriedma.

LGTM on the polly bits.

The polly output is changing because polly strips "zext" operations off SCEV expressions, but doesn't touch extensions inside a SCEV expression. The updated test still tests what it's supposed to test.

This revision is now accepted and ready to land.Jun 21 2018, 2:15 PM
This revision was automatically updated to reflect the committed changes.