[Nestmates] RFR: 8187360) [Nestmates] Remove invokespecial changes

Karen Kinnear karen.kinnear at oracle.com
Thu Sep 14 02:15:08 UTC 2017


Would it work for you to make a single webrev from the current state of your nest mate repo relative
to the parent? I am trying to review relative to the spec and found this changeset and the last one, but
not the one before.

That would it make it much easier for me to review all together.

thanks very much,
Karen

> On Sep 10, 2017, at 10:15 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> On 8/09/2017 4:20 PM, David Holmes wrote:
>> webrev: http://cr.openjdk.java.net/~dholmes/8187360/webrev/
> 
> I was asked for a webrev showing diffs to jdk10/hs code to confirm things were reverted correctly:
> 
> http://cr.openjdk.java.net/~dholmes/8187360/webrev.jdk10/
> 
> The changes to verifier.cpp are indentation only.
> 
> Thanks,
> David
> 
>> The initial proposal was extending the use of invokespecial to allow it to be used for private nestmate method invocations. To support that changes had to be made to the verifier. After JVMLS it was decided that we would use invokevirtual and invokeinterface (as appropriate) as the specification changes needed are much simpler. The switch to that approach was done under JDK-8187221 and JDK-8186763. Now we can remove the invokespecial changes that were initially put in place.
>> Thanks,
>> David



More information about the valhalla-dev mailing list