RFR: 8230851 Interpreter could avoid calling the runtime for defaultvalue
frederic.parain at oracle.com
Wed Sep 11 14:13:49 UTC 2019
Thank you for the review.
I’ve moved the verify_oop() as suggested:
I’ve made the CR x86 specific, and also added [x86] to the summary.
Do you need a more specific label?
> 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.
> 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.
> 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,
More information about the valhalla-dev