hg: valhalla/valhalla/hotspot: Fix and cleanup in vwithfield

Karen Kinnear karen.kinnear at oracle.com
Wed Jun 7 21:04:35 UTC 2017


Frederic,

Looks good. Thank you for the cleanup.

One question please - why do you add type2size(T_VALUETYPE) * stackElementSize
when you return the size of the new value type result?

thanks,
Karen

> On Jun 7, 2017, at 3:41 PM, Frederic Parain <frederic.parain at oracle.com> wrote:
> 
> Fixing a bug in vwithfield leading to buggy stack management.
> Thanks to Mr Simms for having spotted and identify the issue.
> I took the opportunity to do a clean up of the vwithfield method,
> removing raw pointer arithmetic on the expression stack, using
> the frame API instead.
> 
> Fred
> 
> 
>> On Jun 7, 2017, at 15:39, frederic.parain at oracle.com wrote:
>> 
>> Changeset: b03e14ea3e9b
>> Author:    fparain
>> Date:      2017-06-07 15:38 -0400
>> URL:       http://hg.openjdk.java.net/valhalla/valhalla/hotspot/rev/b03e14ea3e9b
>> 
>> Fix and cleanup in vwithfield
>> 
>> ! src/share/vm/interpreter/interpreterRuntime.cpp
>> 
> 



More information about the valhalla-dev mailing list