[11u] RFR: Missing intrinsics for Math.ceil, floor, rint

[Hohensee, Paul]

Done and pushed.

Thanks,
Paul

-----Original Message-----
From: "Astigeevich, Evgeny" <eastig at amazon.co.uk>
Date: Thursday, May 6, 2021 at 4:23 AM
To: "Hohensee, Paul" <hohensee at amazon.com>, "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
Subject: Re: [11u] RFR: Missing intrinsics for Math.ceil, floor, rint

Hi Paul,

Thanks for reviewing.

Could you please sponsor this and 8231713?

Thanks,
Evgeny

On 05/05/2021, 21:48, "Astigeevich, Evgeny" <eastig at amazon.com> wrote:



    On 29/04/2021, 18:53, "Hohensee, Paul" <hohensee at amazon.com> wrote:

        Lgtm, but note that the JBS issue says it needs 8231713 (which is trivial) as a follow-up.

        Thanks,
        Paul

        -----Original Message-----
        From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> on behalf of "Astigeevich, Evgeny" <eastig at amazon.co.uk>
        Date: Wednesday, April 28, 2021 at 2:59 PM
        To: "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
        Subject: [11u] RFR 8226721: Missing intrinsics for Math.ceil, floor, rint

        Hi,

        Please review the backport of JDK-8226721to 11u.

        Bug: https://bugs.openjdk.java.net/browse/JDK-8226721
        Original patch: https://hg.openjdk.java.net/jdk/jdk/rev/6fc57e391539

        The original patch does not apply cleanly to 11u. The update to the patch is to reposition the changes to five files:

        src/hotspot/cpu/x86/x86.ad
        src/hotspot/share/opto/library_call.cpp
        src/hotspot/share/opto/superword.cpp
        src/hotspot/share/opto/vectornode.cpp
        src/hotspot/share/opto/vectornode.hpp

        The content of the patch is not changed.

        11u webrev: http://cr.openjdk.java.net/~eastigeevich/8226721/webrev.00/

        Tested: Amazon Linux 2 x86_64, tier1, tier2, test/hotspot/jtreg/compiler/c2/cr6340864/TestDoubleVect.java
        Note: There is a failing test: sun/security/pkcs11/Secmod/AddTrustedCert.java. It fails without the patch as well. Details: https://bugs.openjdk.java.net/browse/JDK-8232153

        Thanks,
        Evgeny Astigeevich