RFR: 8358556: Assert when running with -XX:-UseLibmIntrinsic [v4]
    Sandhya Viswanathan 
    sviswanathan at openjdk.org
       
    Mon Jun  9 18:34:52 UTC 2025
    
    
  
On Sat, 7 Jun 2025 00:57:25 GMT, Mohamed Issa <duke at openjdk.org> wrote:
>> This change fixes unwanted calls to the cbrt and tanh intrinsics on x86 when they are explicitly disabled by flags such as _-XX:-UseLibmIntrinsic_ or _-XX:-InlineIntrinsics_.
>
> Mohamed Issa has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add extra check for InlineIntrinscs flag
src/hotspot/share/classfile/vmIntrinsics.cpp line 319:
> 317:   case vmIntrinsics::_dtanh:
> 318:   case vmIntrinsics::_dcbrt:
> 319:     if (!InlineMathNatives || !InlineIntrinsics) return true;
You also need to add check for UseLibmIntrinsic here:
#ifdef AMD64
    if (!UseLibmIntrinsic)  return true;
#endif
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25680#discussion_r2136250496
    
    
More information about the hotspot-runtime-dev
mailing list