RFR: 8345219: C2: x86_64 should not go to interpreter stubs for NaNs handling [v3]
Emanuel Peter
epeter at openjdk.org
Wed Dec 4 15:11:42 UTC 2024
On Wed, 4 Dec 2024 14:48:55 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> test/hotspot/jtreg/compiler/c2/irTests/TestFPConversion.java line 33:
>>
>>> 31: * @summary Test that code generation for FP conversion works as intended
>>> 32: * @library /test/lib /
>>> 33: * @requires os.arch != "x86" & os.arch != "i386"
>>
>> It would have been preferrable to add this to the IR rule, so the test still runs elsewhere.
>> What about `aarch64`?
>
> For all practical purposes, the test will run everywhere, really. It would be skipped only on `IA32`, which is going away. This `@requires` serves as the concession to `IA32` that is still in the tree.
Ah, I read the requires wrong. All good.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22446#discussion_r1869731836
More information about the core-libs-dev
mailing list