RFR: 8230851 Interpreter could avoid calling the runtime for defaultvalue

Dmitry Samersoff dms at samersoff.net
Wed Sep 11 13:16:58 UTC 2019


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