Request for reviews (S): 7054211: No loop unrolling done in jdk7b144 for a test update() while loop

[Vladimir Kozlov]

Thank you, Tom

I updated comments.

Vladimir

Tom Rodriguez wrote:
> Though it might be nice to correct the comment to say that it helps crc32 and not CaffeineMark.
> 
> tom
> 
> On Sep 7, 2011, at 9:29 AM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7054211/webrev
>>
>> 7054211: No loop unrolling done in jdk7b144 for a test update() while loop
>>
>> It is anti-delta of changes which removed code for CaffeineMark. A perf regression of approximately 13.5% was observed in benchmark with a pure java crc32 implementation which use XORs after that changes.
>>
>> Our refworkload benchmarks are not affected by this fix (and that is why did original changes).
>>
>> Thanks,
>> Vladimir
>