Request for reviews (L) (resending): 6827605 and 6875866 for HS17
Vladimir.Kozlov at Sun.COM
Mon Sep 14 11:57:07 PDT 2009
Tom Rodriguez wrote:
> looks good.
> On Sep 11, 2009, at 10:10 AM, Vladimir Kozlov wrote:
>> I still need reviews for this (and I merged it with latest sources).
>> Fixed 6827605: new String intrinsics may prevent EA scalar replacement
>> Fixed 6875866: Intrinsic for String.indexOf() is broken on x86 with
>> New String.indexOf SSE4.2 intrinsic prevents EA scalar replacement
>> of String object clone in the famous hot jbb code in Orderline.process:
>> stockData.indexOf("ORIGINAL") (6827605).
>> Also String.indexOf SSE4.2 code starts string rescanning from
>> a wrong place after failing to match a part of substring (6875866).
>> Modify String intrinsic methods to pass char pointers and
>> counters instead of string oops to allow EA eliminate
>> non-escaping string objects. Move some checks from
>> mach nodes encoding into ideal graph.
>> Move common assembler code into MacroAssembler.
>> Add memory edge to AryEqNode.
>> Fix 6875866.
>> Reviewed by:
>> Fix verified (y/n): y, refworkload
>> Other testing:
>> JPRT, CTW
More information about the hotspot-compiler-dev