8226721: Missing intrinsics for Math.ceil, floor, rint

[Bhateja, Jatin]

Hi Eric,

Thanks for pointing this, updated patch is kept at following link.
http://cr.openjdk.java.net/~jbhateja/8226721/webrev.06/webrev/

Regards,
Jatin

> -----Original Message-----
> From: hotspot-compiler-dev <hotspot-compiler-dev-
> bounces at openjdk.java.net> On Behalf Of Eric Caspole
> Sent: Wednesday, September 4, 2019 1:07 AM
> To: hotspot-compiler-dev at openjdk.java.net
> Subject: Re: 8226721: Missing intrinsics for Math.ceil, floor, rint
> 
> Hi Jatin,
> I have a few comments on the JMH in the webrev:
> 
> 1 - Make the TESTSIZE into a JMH Param, so it will be more versatile:
> 
>    public final int TESTSIZE = 1024;
> 
> becomes:
> 
>    @Param({"1024"})
>    public int testSize;
> 
> 
> 2 - the DspecialVals is not used at all
> 
>    public final double[] DspecialVals = {
>        0.0, -0.0, Double.NaN, Double.NEGATIVE_INFINITY,
> Double.POSITIVE_INFINITY};
> 
> 
> 
> Thanks for including the JMH from start with your webrev.
> Eric
> 
> 
> On 9/3/19 05:41, Bhateja, Jatin wrote:
> > Hi All,
> >
> > Please find a patch with the following changes:-
> > 1) Intrincifiation for Math.ceil/floor/rint.
> > 2) Auto-vectorizer handling.
> >
> > JBS: https://bugs.openjdk.java.net/browse/JDK-8226721
> > Webrev: http://cr.openjdk.java.net/~jbhateja/8226721/webrev.05
> >
> > Kindly review it.
> >
> > Best Regards,
> > Jatin
> >