RFR: JDK-8223346 - Update Graal

dean.long at oracle.com dean.long at oracle.com
Tue May 14 21:03:28 UTC 2019

I suggest doing the tiers 1-4 testing separate from the tier5+ testing, 
to reduce noise.

There is a fix for CheckGraalIntrinsics coming to upstream Graal. When 
Jesper merges the overwritten changes, he could include that fix as 
well, so that compiler/graalunit/HotspotTest.java passes.

The HeapMonitorStatArrayCorrectnessTest failure should have been fixed 
by JDK-8223441, unless Jesper's test repo is out of date.


On 5/14/19 12:32 PM, Vladimir Kozlov wrote:
> Changes seems fine but I am not comfortable about tests results. There 
> are a lot of timeouts again but there are many graalunit tests failures.
> This time you have to apply overwritten diffs after merge (if we 
> decide to push it) - these changes are not in Graal master repo yet.
> Thanks,
> Vladimir
> On 5/13/19 5:19 PM, jesper.wilhelmsson at oracle.com wrote:
>> Hi,
>> Please review the patch to integrate recent Graal changes into OpenJDK.
>> Graal tip to integrate: 6a18d9ddacd8eecb0ae4877f687e171889939c0d
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8223346
>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8223346/webrev.00/
>> This integration did overwrite changes already in place in OpenJDK. 
>> The diff has been attached to the umbrella bug.
>> Thanks,
>> /Jesper

More information about the graal-dev mailing list