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

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Wed Nov 19 10:30:30 UTC 2014


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
>


More information about the mlvm-dev mailing list