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

[Nils Eliasson]

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