RFR: JDK-8218074 - Update Graal

jesper.wilhelmsson at oracle.com jesper.wilhelmsson at oracle.com
Mon Mar 11 00:56:23 UTC 2019

Hi Vladimir,

I followed the given instructions and used mx to create the diff:

mx --java-home=$JAVA_HOME updategraalinopenjdk cleanjdk/open 12

I assume JDK-8215322 is in there because it was actually pushed to OpenJDK and now it was overwritten (with the same diff). It seems the mx script do not realize that it's the same cahange. The diff and patch and all changes made was done by the new script, so no extra human labor has been put into it at this point. If a human is required to modify the overwritten-diff I suggest that is part of the review step (as now) and that a new patch is simply added to the subtask.

I have removed JDK-8215322 from the diff now.


> On 9 Mar 2019, at 18:42, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> Hi Jesper,
> Thank you for doing this.
> I looked on 8220387 to see what changes your webrev does not have and it is strange. You should not have such big difference. How you prepared overwritten-diffs.txt?
> For example, JAOTC changes "8215322: add @file support to jaotc" are in overwritten-diffs.txt. But changeset is listed in the 8218074 Graal changelog list [1].
> Thanks,
> Vladimir
> [1] 8433917 Wed Dec 19 16:20:00 2018 -0800 Igor Ignatyev [GR-13142] Add at-file support to jaotc.
> On 3/9/19 12:46 AM, jesper.wilhelmsson at oracle.com wrote:
>> Hi,
>> Please review the patch to integrate the latest Graal changes into OpenJDK.
>> Graal tip to integrate: db79f81716886b7883370cd6ea1bbf5c42966fa5
>> JBS duplicates fixed by this integration:
>> https://bugs.openjdk.java.net/browse/JDK-8217161
>> https://bugs.openjdk.java.net/browse/JDK-8218698
>> https://bugs.openjdk.java.net/browse/JDK-8218859
>> JBS duplicates deferred to the next integration:
>> https://bugs.openjdk.java.net/browse/JDK-8214947
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8218074
>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8218074/webrev.00/
>> This integration did overwrite changes already in place in OpenJDK. A subtask was created to hold the diff: https://bugs.openjdk.java.net/browse/JDK-8220387
>> Thanks,
>> /Jesper

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Message signed with OpenPGP
URL: <https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20190311/5e678167/signature.asc>

More information about the hotspot-compiler-dev mailing list