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

[Chris Lattner]

On Sep 8, 2010, at 1:48 PM, Owen Anderson wrote:

> 
> 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:

Sounds great, thanks Owen!

-Chris