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