[11u] RFR (S) 8214922: Aarch64: Add vectorization support for fmin/fmax

[Dmitry Chuyko]

Hi,

Volker, Christoph, thanks for the catch. I included formatting changes 
into the patch.

webrev: http://cr.openjdk.java.net/~dchuyko/8214922/webrev.11u.01/

Combined changes were re-tested and re-measured.

-Dmitry

On 3/10/21 1:57 PM, Langer, Christoph wrote:
> Hi,
>
> I agree, the format changes from 8241911 should be added to this backport.
>
> Best regards
> Christoph
>
>> -----Original Message-----
>> From: Volker Simonis <volker.simonis at gmail.com>
>> Sent: Montag, 8. März 2021 13:15
>> To: Dmitry Chuyko <dmitry.chuyko at bell-sw.com>
>> Cc: jdk-updates-dev at openjdk.java.net; aarch64-port-
>> dev at openjdk.java.net; Langer, Christoph <christoph.langer at sap.com>
>> Subject: Re: [11u] RFR (S) 8214922: Aarch64: Add vectorization support for
>> fmin/fmax
>>
>> Hi Dmitry,
>>
>> in general, the downport looks good to me.
>>
>> I only wonder if we shouldn't integrate the format changes from
>> 8241911 which were skipped in December when 8241911 was downported
>> [1,2] because at that time this change wasn't in 11u. I'm fine either
>> way and leave the final decision to the 11u maintainers.
>>
>> Thank you and best regards,
>> Volker
>>
>> [1] https://mail.openjdk.java.net/pipermail/jdk-updates-dev/2020-
>> December/004474.html
>> [2] https://mail.openjdk.java.net/pipermail/jdk-updates-dev/2020-
>> December/004498.html
>>
>> On Thu, Feb 18, 2021 at 9:27 AM Dmitry Chuyko <dmitry.chuyko at bell-
>> sw.com> wrote:
>>> Hello,
>>>
>>> Original RFE: https://bugs.openjdk.java.net/browse/JDK-8214922
>>>
>>> Original patch applies almost cleanly except 2 added lines in
>>> src/hotspot/share/opto/vectornode.cpp because of missing part of
>> context
>>> (JDK-8214751 not ported), they are added manually.
>>>
>>> 11u webrev:
>> http://cr.openjdk.java.net/~dchuyko/8214922/webrev.11u.00/
>>> Testing: tier1, tier2, TestFpMinMaxIntrinsics.
>>>
>>> --
>>> Thanks,
>>> -Dmitry
>>>