On Mon, Feb 14, 2011 at 22:06, Noah Misch <noah@leadboat.com> wrote:
> On Sun, Feb 13, 2011 at 12:41:11PM -0600, Kevin Grittner wrote:
>> Unpatched:
>> real 17m24.171s
>> real 16m52.892s
>> real 16m40.624s
>> real 16m41.700s
>>
>> Patched:
>> real 15m56.249s
>> real 15m47.001s
>> real 15m3.018s
>> real 17m16.157s
>>
>> Since you said that a cursory test, or no test at all, should be
>> good enough given the low risk of performance regression, I didn't
>> book a machine and script a large test run, but if anyone feels
>> that's justified, I can arrange something.
>
> Based on this, I've taken the liberty of marking the patch Ready for Committer.
Thank you very much for performance testing and reviewing!
The result is interesting because I didn't intend performance optimization.
At least no performance regression is enough for the purpose.
--
Itagaki Takahiro