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

[Nils Eliasson]

Yes, I will sponsor it as soon as I have reviewed it.

// Nils

On 2019-09-18 17:49, Bhateja, Jatin wrote:
> 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