Integrated: 8329355: Test compiler/c2/irTests/TestIfMinMax.java fails on RISC-V
Fei Yang
fyang at openjdk.org
Wed Apr 3 07:42:04 UTC 2024
On Sat, 30 Mar 2024 08:49:00 GMT, Fei Yang <fyang at openjdk.org> wrote:
> Please review this small change fixing an IR matching failure on linux-riscv platform.
>
> JDK-8324655 tries to identify min/max patterns in CMoves and transform them into Min and Max nodes.
> But architectures like RISC-V doesn't have support of conditional moves at the ISA level for now.
> So we set ConditionalMoveLimit parameter to 0 for this platform and conditionals moves are emulated
> with normal compare and branch instructions instead [1]. This is why the IR matching test added by
> JDK-8324655 fails on this platform. A simple way to fix this would be skip this test for this case.
>
> [1] https://github.com/openjdk/jdk/blob/master/src/hotspot/cpu/riscv/riscv.ad#L9775
This pull request has now been integrated.
Changeset: 16b842af
Author: Fei Yang <fyang at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/16b842af8edd10c4071eec98caf838a2f6c49746
Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
8329355: Test compiler/c2/irTests/TestIfMinMax.java fails on RISC-V
Reviewed-by: jkarthikeyan, thartmann, chagedorn
-------------
PR: https://git.openjdk.org/jdk/pull/18558
More information about the hotspot-compiler-dev
mailing list