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

[Bhateja, Jatin]

Hi Nils,

Thanks a lot for testing the patch.
I have uploaded the re-based patch at following link.

http://cr.openjdk.java.net/~jbhateja/8226721/webrev.07/

It will be helpful if you can sponsor this patch.

Best Regards,
Jatin

> -----Original Message-----
> From: hotspot-compiler-dev <hotspot-compiler-dev-
> bounces at openjdk.java.net> On Behalf Of Nils Eliasson
> Sent: Thursday, September 12, 2019 5:07 PM
> To: hotspot-compiler-dev at openjdk.java.net
> Subject: Re: 8226721: Missing intrinsics for Math.ceil, floor, rint
> 
> Testing passed.
> 
> // Nils
> 
> On 2019-09-11 21:59, Nils Eliasson wrote:
> > Hi Jatin,
> >
> > Sorry for the delay. I have started testing now.
> >
> > Best regards,
> >
> > Nils
> >
> > On 2019-09-09 18:45, Bhateja, Jatin wrote:
> >> Hi Nils,
> >>
> >> I have taken care of other C2 specific review comments over this
> >> patch, following is the link to updated patch.
> >> http://cr.openjdk.java.net/~jbhateja/8226721/webrev.06/webrev/
> >>
> >> It will be helpful if you can run these though your tests.
> >>
> >> Best Regards,
> >> Jatin
> >>
> >>> -----Original Message-----
> >>> From: hotspot-compiler-dev <hotspot-compiler-dev-
> >>> bounces at openjdk.java.net> On Behalf Of Bhateja, Jatin
> >>> Sent: Wednesday, September 4, 2019 7:17 AM
> >>> To: Nils Eliasson <nils.eliasson at oracle.com>; hotspot-compiler-
> >>> dev at openjdk.java.net
> >>> Subject: Re: 8226721: Missing intrinsics for Math.ceil, floor, rint
> >>>
> >>> Hi Nils,
> >>>
> >>> I will convert new vector patterns added by this patch to generic
> >>> operands patterns once this lands into mainline, will update the
> >>> other patch.
> >>>
> >>> Regards,
> >>> Jatin
> >>>
> >>>
> >>> -------- Original message --------
> >>> From: Nils Eliasson <nils.eliasson at oracle.com>
> >>> Date: 04/09/2019 00:55 (GMT+05:30)
> >>> To: hotspot-compiler-dev at openjdk.java.net
> >>> Subject: Re: 8226721: Missing intrinsics for Math.ceil, floor, rint
> >>>
> >>> Hi Jatin,
> >>>
> >>> Thanks for adding these.
> >>>
> >>> Does this patch conflict with your patch that adds generic vector
> >>> operands?
> >>> Do you have any preferred order?
> >>>
> >>> I can take them for a spin through our testing.
> >>>
> >>> Best regards,
> >>>
> >>> Nils
> >>>
> >>>
> >>> On 2019-09-03 11: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