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

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Sep 7 09:37:33 PDT 2011


Thank you, Tom

I will update 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



More information about the hotspot-compiler-dev mailing list