This is an archive of the discontinued LLVM Phabricator instance.

[SCEV] Introduce add operation inlining limit
ClosedPublic

Authored by dfukalov on Jan 17 2017, 9:53 AM.

Details

Summary

Inlining in getAddExpr() can cause abnormal computational time in some cases. new parameter -scev-addops-inline-threshold is intruduced with default value 500.

Diff Detail

Repository
rL LLVM

Event Timeline

dfukalov created this revision.Jan 17 2017, 9:53 AM
dfukalov retitled this revision from [SCEV] Add add operation inlining limit to [SCEV] Introduce add operation inlining limit.Jan 18 2017, 2:44 AM
sanjoy accepted this revision.Jan 25 2017, 5:30 PM

LGTM

I generally don't like thresholds, but I can't immediately think of anything better than AddOpsInlineThreshold.

This revision is now accepted and ready to land.Jan 25 2017, 5:30 PM
This revision was automatically updated to reflect the committed changes.