8226721: Missing intrinsics for Math.ceil, floor, rint
Nils Eliasson
nils.eliasson at oracle.com
Thu Sep 12 11:36:38 UTC 2019
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
More information about the hotspot-compiler-dev
mailing list