RFR: 8358556: Assert when running with -XX:-UseLibmIntrinsic [v2]

Mohamed Issa duke at openjdk.org
Fri Jun 6 23:33:05 UTC 2025


On Fri, 6 Jun 2025 22:58:24 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> Mohamed Issa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Move UseLibmIntrinsic flag to x86 portion of code base to avoid build breaks
>
> src/hotspot/share/classfile/vmIntrinsics.cpp line 263:
> 
>> 261:     switch (id) {
>> 262:     case vmIntrinsics::_dtanh:
>> 263:     case vmIntrinsics::_dcbrt:
> 
> You could also move _floatToFloat16 and _float16ToFloat to here from the switch below to make it consistent.

Ok, I moved them.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25680#discussion_r2133072865


More information about the hotspot-runtime-dev mailing list