[llvm-commits] [llvm] r113306 - /llvm/trunk/lib/Transforms/Scalar/LoopUnrollPass.cpp

[Owen Anderson]

On Sep 7, 2010, at 4:15 PM, Owen Anderson wrote:

> Author: resistor
> Date: Tue Sep  7 18:15:30 2010
> New Revision: 113306
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=113306&view=rev
> Log:
> Add a separate unrolling threshold when the current function is being optimized for size.
> The threshold value of 50 is arbitrary, and I chose it simply by analogy to the inlining thresholds, where
> the baseline unrolling threshold is slightly smaller than the baseline inlining threshold.  This could
> undoubtedly use some tuning.

Actually, after researching this, it looks like 50 is a pretty decent threshold choice, as evidenced by the attached data from SPEC2000/2006:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: CodeSize.png
Type: image/png
Size: 27210 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100908/6bd4b459/attachment.png>
-------------- next part --------------


--Owen