RFR: 8302976: C2 intrinsification of Float.floatToFloat16 and Float.float16ToFloat yields different result than the interpreter

[Sandhya Viswanathan]

On Tue, 7 Mar 2023 01:59:25 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> src/hotspot/cpu/x86/stubGenerator_x86_64.cpp line 3931:
>> 
>>> 3929:     // For results consistency both intrinsics should be enabled.
>>> 3930:     if (vmIntrinsics::is_intrinsic_available(vmIntrinsics::_float16ToFloat) &&
>>> 3931:         vmIntrinsics::is_intrinsic_available(vmIntrinsics::_floatToFloat16)) {
>> 
>> Should this also check for InlineIntrinsics?
>
> `vmIntrinsics::disabled_by_jvm_flags()` checks `InlineIntrinsics`. See `vmIntrinsics.cpp` changes.

Yes you are right.

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

PR: https://git.openjdk.org/jdk/pull/12869