Return value type fields in registers

Roland Westrelin rwestrel at redhat.com
Wed May 3 08:42:05 UTC 2017


Hi Tobias,

Thanks for looking at this.

> templateInterpreterGenerator_x86.cpp
> - What is 'skip' used for?

It's useless indeed.

> deoptimization.cpp
> - you can merge the check in line 232 with the check in line 252

It's not that simple: line 245 changes the save_oop_result value so the
flag has to be retested.

> - 'return_values' is a bit misleading because it contains Handles for
> oop fields or the oop return and not all returned values. Maybe rename
> to 'return_oops' or 'return_handles'.

ok.

> sharedRuntime.cpp
> - line 3176, missing whitespace

Right.

I have a revised change coming that will take care of your comments.

Roland.



More information about the valhalla-dev mailing list