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

[Bhateja, Jatin]

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