[9, 8u40] RFR (M): 8063135: Enable full LF sharing by default

[Aleksey Shipilev]

Looks good now.

-Aleksey.

On 11/19/2014 01:30 PM, Vladimir Ivanov wrote:
> Hm, I remember I fixed that long time ago... Seems like I chose a stale
> patch. Sorry for that. Updated webrev in place.
> 
> Best regards,
> Vladimir Ivanov
> 
> On 11/19/14, 3:38 AM, Aleksey Shipilev wrote:
>> On 11/18/2014 11:23 PM, Vladimir Ivanov wrote:
>>> http://cr.openjdk.java.net/~vlivanov/8063135/webrev.00/
>>> https://bugs.openjdk.java.net/browse/JDK-8063135
>>
>> Broken array index here:
>>
>>   69         TRACE_METHOD_LINKAGE      = (Boolean) values[3];
>>   70         COMPILE_THRESHOLD         = (Integer) values[5];
>>
>> Also, how does it pass the tests? It should fail with AIOBE during
>> MethodHandleStatics::<clinit>, since
>>
>>   53         final Object[] values = new Object[7];
>>   ...
>>   72         PROFILE_LEVEL             = (Integer) values[7];
>>
>> Thanks,
>> -Aleksey.
>>
>>
>>
>> _______________________________________________
>> mlvm-dev mailing list
>> mlvm-dev at openjdk.java.net
>> http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
>>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/mlvm-dev/attachments/20141119/e450c658/signature-0001.asc>