RFR(S): 8188809: [MVT] C2 support for interpreter buffering of value types

Tobias Hartmann tobias.hartmann at oracle.com
Wed Nov 8 15:15:32 UTC 2017


Hi Roland,

thanks for the review!

On 08.11.2017 15:18, Roland Westrelin wrote:
>> http://cr.openjdk.java.net/~thartmann/8188809/webrev.00/
> 
> Why the change to sharedRuntime_x86_64.cpp?

I've just checked and it does not seem necessary. Not sure why I added it.

> The rest looks reasonable to me.

Thanks, I'll push without the change in sharedRuntime_x86_64.cpp.

Best regards,
Tobias



More information about the valhalla-dev mailing list