RFR: JDK-8218074 - Update Graal
doug.simon at oracle.com
Mon Mar 11 09:45:29 UTC 2019
> On 11 Mar 2019, at 01:56, jesper.wilhelmsson at oracle.com wrote:
> 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.
Correct. To detect that we would have to add logic for diffs of diffs and the complexity didn’t seem worth it. I’d more more than happy to review a smart person’s attempt at adding it though ;-)
> 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 .
>>  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:
>>> Please review the patch to integrate the latest Graal changes into OpenJDK.
>>> Graal tip to integrate: db79f81716886b7883370cd6ea1bbf5c42966fa5
>>> JBS duplicates fixed by this integration:
>>> JBS duplicates deferred to the next integration:
>>> 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
More information about the hotspot-compiler-dev