RFR: 8276151: AArch64: Incorrect result for double to int vector conversion [v2]

[Ningsheng Jian]

On Thu, 4 Nov 2021 09:21:43 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> Ningsheng Jian has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review comments from Andrew.
>
> src/hotspot/cpu/aarch64/aarch64_neon.ad line 518:
> 
>> 516:   ins_encode %{
>> 517:     __ ins(as_FloatRegister($dst$$reg), __ D, as_FloatRegister($src$$reg), 0, 1);
>> 518:     // Converting from double to int directly follows Java specification.
> 
> Suggestion:
> 
>     // We can't use fcvtzs(vector, integer) here because we need saturation arithmetic. See JDK-8276151.

Thank you @theRealAph ! Updated.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6247