RFR: 8230851 Interpreter could avoid calling the runtime for defaultvalue
Frederic Parain
frederic.parain at oracle.com
Wed Sep 11 14:13:49 UTC 2019
Dmitry,
Thank you for the review.
I’ve moved the verify_oop() as suggested:
http://cr.openjdk.java.net/~fparain/opt_int/webrev.01/index.html
I’ve made the CR x86 specific, and also added [x86] to the summary.
Do you need a more specific label?
Regards,
Fred
> On Sep 11, 2019, at 09:16, Dmitry Samersoff <dms at samersoff.net> wrote:
>
> Hello Frederic,
>
> Looks good to me.
>
> You may consider to move
>
> __ verify_oop(rax);
>
> after done, to save few instructions.
>
> PS:
>
> Could you add a label (something like x86_specific) to the CR to
> simplify further porting.
>
> We discussed this approach with IoI and it may be a time to start
> adopting it.
>
> -Dmitry
>
> On 11.09.19 15:20, Frederic Parain wrote:
>> Please this review this small change in the interpreter adding
>> a fast path to defaultvalue.
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8230851
>> Webrev: http://cr.openjdk.java.net/~fparain/opt_int/webrev.00/index.html
>>
>> Thank you,
>>
>> Fred
>>
More information about the valhalla-dev
mailing list