RFR: 8350835: C2 SuperWord: assert/wrong result when using Float.float16ToFloat with byte instead of short input [v3]
Emanuel Peter
epeter at openjdk.org
Mon Mar 17 16:29:41 UTC 2025
On Sat, 15 Mar 2025 00:56:25 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:
>> Float.float16ToFloat generates wrong vectorized code in product build and asserts in fastdebug/debug when argument is of type byte, int, or long array. The short term solution is to not auto vectorize in these cases.
>>
>> Review comments are welcome.
>>
>> Best Regards,
>> Sandhya
>
> Sandhya Viswanathan has updated the pull request incrementally with one additional commit since the last revision:
>
> Review comments from Emanuel
Tests are passing. Approved.
@sviswa7 Thanks for the work 😊
-------------
Marked as reviewed by epeter (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23939#pullrequestreview-2691176744
More information about the hotspot-compiler-dev
mailing list