RFR: JDK-8225497 - Update Graal

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jun 26 16:33:40 UTC 2019


Looks good.

I looked on testing results. We still hit gcbasher failures but it is not new and we will work on it.
Other failures I known timeout or other known bugs.

I think we can push this into jdk 13.

Thanks,
Vladimir

On 6/25/19 6:46 PM, jesper.wilhelmsson at oracle.com wrote:
> This is a webrev based on JDK 13: http://cr.openjdk.java.net/~jwilhelm/8225497/webrev.01/ <http://cr.openjdk.java.net/~jwilhelm/8225497/webrev.01/>
> I have restarted the testing with this merge.
> /Jesper
> 
>> On 26 Jun 2019, at 01:15, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>
>> In general changes looks good.
>>
>> But changes are and the testing are done with jdk/jdk as base. We need this 'Update Graal' in JDK 13.
>>
>> I looked on testing results and they look good so far (still running). I linked existing bugs for some failures. And other failures seem were present before.
>>
>> I checked overwritten-diffs.txt and you don't need to apply it. Changes in this 'Update Graal' includes them. And again, for JDK 13 it could be different.
>>
>> Thanks,
>> Vladimir
>>
>> On 6/25/19 2:21 PM, jesper.wilhelmsson at oracle.com wrote:
>>> Oops. Fixed.
>>> /Jesper
>>>> On 25 Jun 2019, at 21:57, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>>>
>>>> Hi Jesper
>>>>
>>>> Webrev link is broken.
>>>>
>>>> Vladimir
>>>>
>>>> On 6/25/19 12:51 PM, jesper.wilhelmsson at oracle.com wrote:
>>>>> Hi,
>>>>> Please review the patch to integrate recent Graal changes into OpenJDK.
>>>>> Graal tip to integrate: ea9d5c4b76a4ba95f80945970ad8489db52344ff
>>>>> JBS duplicates fixed by this integration:
>>>>> https://bugs.openjdk.java.net/browse/JDK-8223924
>>>>> https://bugs.openjdk.java.net/browse/JDK-8221514
>>>>> https://bugs.openjdk.java.net/browse/JDK-8221577
>>>>> https://bugs.openjdk.java.net/browse/JDK-8224254
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8225497
>>>>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8225497/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 hotspot-compiler-dev mailing list