RFR(L) 8205824: Update Graal

Nils Eliasson nils.eliasson at oracle.com
Fri Jun 29 13:40:00 UTC 2018



On 2018-06-27 22:45, Tom Rodriguez wrote:
> https://bugs.openjdk.java.net/browse/JDK-8205824
> http://cr.openjdk.java.net/~never/8205824/webrev/index.html
>
> This combines a general Graal update with the fix for 
> https://bugs.openjdk.java.net/browse/JDK-8204855.  This general Graal 
> update also fixes JDK-8204914 and includes a few other fixes from 
> Graal.  Unfortunately the webrev is fairly noisy because of blank 
> lines inserted because of copyright adjustment and the addition of 
> @Override annotations in JVMCI.
>
> 8204855 is a set of changes to the JVMCI API which are required to 
> support the libgraal effort.  Primarily this is about making a strong 
> distinction between runtime and compiler objects and state and hiding 
> anything which lets you bridge that boundary without be careful.  
> Mostly it doesn't change the way JVMCI works apart from some changes 
> to the HotSpotSpeculationLog implementation and the associated changes 
> in Graal.  These changes are binary incompatible with previous 
> versions of JVMCI.  Given the limitations of webrev I can't easily 
> provide a webrev which shows just this change.

There have been several examples of incremental webrevs for large 
changes recently. It's impossible to have an opinion on the jvmci 
changes given this webrev.

// Nils

>
> The bits were tested with mach5 
> builds-tier1,jdk-tier1,hs-tier1,hs-tier2,hs-tier3,hs-tier4-graal,hs-tier5-graal 
> and hs-tier6-graal.  The details are in the bug report.
>
> I'm happy to explain anything about changes the JVMCI and why they 
> were required.  The JVMCI API updates have been tested with JDK8 Graal 
> and will land in the next few days.
>
> tom



More information about the hotspot-compiler-dev mailing list