This is an archive of the discontinued LLVM Phabricator instance.

[LoopUnrollAnalyzer] Don't treat gep-instructions with simplified offset as simplified.
ClosedPublic

Authored by mzolotukhin on Feb 17 2016, 6:51 PM.

Details

Summary

Currently we consider such instructions as simplified, which is incorrect,
because if their user isn't simplified, we can't actually simplify them too.
This biases our estimates of profitability: for instance the analyzer expects
much more gains from unrolling memcpy loops than there actually are.

Diff Detail

Event Timeline

mzolotukhin retitled this revision from to [LoopUnrollAnalyzer] Don't treat gep-instructions with simplified offset as simplified..
mzolotukhin updated this object.
mzolotukhin added reviewers: chandlerc, hfinkel.
mzolotukhin added a subscriber: llvm-commits.
chandlerc accepted this revision.May 12 2016, 3:31 PM
chandlerc edited edge metadata.

LGTM!

This revision is now accepted and ready to land.May 12 2016, 3:31 PM
This revision was automatically updated to reflect the committed changes.