[8u-dev] Request for Approval: Backport of 8184271 and 8184009
David Buck
david.buck at oracle.com
Thu Aug 10 04:50:08 UTC 2017
approved pending successful code review of the JDK 8 version of the
changeset
I am CCing the HotSpot Compiler Development alias. If any Reviewers
include this thread / alias when they approve, no further action will be
required before pushing this change to jdk8u-dev.
The definition we use for "trivial" requires the changeset exported from
JDK 9/10 to be applicable to the JDK 8 forest as-is. This means zero
manual editing. The only exceptions we make are for copyright year and
path un-shuffling. Even if the surrounding lines of code which prevent a
patch from being applied have nothing to do with the change, we require
a new code review for the JDK 8 version of the change.
Cheers,
-Buck
On 2017/08/10 12:02, Tobias Hartmann wrote:
> Hi,
>
> just a reminder. I don't need a dev review for the trivial merge, right?
>
> Thanks,
> Tobias
>
> On 02.08.2017 18:53, Tobias Hartmann wrote:
>> Hi,
>>
>> please approve the backports of the following fixes to JDK 8u:
>>
>> 8184271: Time related C1 intrinsics produce inconsistent results when
>> floating around
>> https://bugs.openjdk.java.net/browse/JDK-8184271
>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-July/026681.html
>>
>> http://cr.openjdk.java.net/~thartmann/8184271/webrev.00/
>> http://hg.openjdk.java.net/jdk10/hs/hotspot/rev/cc2bc9993d64
>>
>> The patch does not apply cleanly but requires a (trivial) merge:
>> http://cr.openjdk.java.net/~thartmann/8184271_8u/webrev.00/
>>
>> 8184009: Missing null pointer check in
>> InterpreterRuntime::update_mdp_for_ret()
>> https://bugs.openjdk.java.net/browse/JDK-8184009
>> http://mail.openjdk.java.net/pipermail/hotspot-dev/2017-July/027514.html
>> http://cr.openjdk.java.net/~thartmann/8184009/webrev.00/
>> http://hg.openjdk.java.net/jdk10/hs/hotspot/rev/56a68e37d23b
>>
>> Both fixes were pushed to JDK 10 and testing showed no problems.
>>
>> Thanks,
>> Tobias
More information about the hotspot-compiler-dev
mailing list