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

Tue Mar 7 18:41:37 UTC 2023

On Tue, 7 Mar 2023 03:00:34 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Other than the minor comments above, the x86 side changes look good to me.
>
> @sviswa7 I update changes based on your comments. Please, look: [9302d4b](https://github.com/openjdk/jdk/pull/12869/commits/9302d4bc00f8f1d8e774a260eb6aacb2d51a2dd4)

> Hi @vnkozlov , There is some discrepancy in results b/w interpreter, C1 and C2 for following case.

And that is fine. Consistency have to be preserved only during one run. Different runs with different flags (with disabled intrinsics, for example) may produce different results.

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

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