[jdk21u-dev] Integrated: 8319970: AArch64: enable tests compiler/intrinsics/Test(Long|Integer)UnsignedDivMod.java on aarch64

Daniel Hu duke at openjdk.org
Thu Aug 29 17:43:32 UTC 2024


On Mon, 12 Aug 2024 22:57:04 GMT, Daniel Hu <duke at openjdk.org> wrote:

> Backport of [JDK-8319970](https://bugs.openjdk.org/browse/JDK-8319970) that enables tests for compiler/intrinsics/Test(Long|Integer)UnsignedDivMod.java on aarch64. Not clean because [RISC-V intrinsics for divideUnsigned and remainderUnsigned were added in JDK22](https://github.com/openjdk/jdk/commit/40a3c35aa5614be4505013d4e92ddb1b556a3622); aarch64 intrinsics for divideUnsigned and remainderUnsigned confirmed to be [released in JDK20](https://bugs.openjdk.org/browse/JDK-8287926).
> 
> This includes backport of [JDK-8319960](https://bugs.openjdk.org/browse/JDK-8319960) because the `UDivModI` or `UDivModL` nodes are supplied from x86_64.ad, thus these node checks are intended for x64 platforms. Although originally intended to fix the test failing for RISC-V intrinsics, without this exclusion the test will also fail for aarch64 platforms, thus it is included as part of the backport.
> 
> Passes affected tests on Linux aarch64 and Mac aarch64.

This pull request has now been integrated.

Changeset: fb728705
Author:    Daniel Hu <costmuch at amazon.com>
Committer: Ben Taylor <btaylor at openjdk.org>
URL:       https://git.openjdk.org/jdk21u-dev/commit/fb72870565681eb67545613cc6c323275198e021
Stats:     10 lines in 2 files changed: 2 ins; 0 del; 8 mod

8319970: AArch64: enable tests compiler/intrinsics/Test(Long|Integer)UnsignedDivMod.java on aarch64
8319960: RISC-V: compiler/intrinsics/TestInteger/LongUnsignedDivMod.java failed with "counts: Graph contains wrong number of nodes"

Reviewed-by: phh
Backport-of: c80e691adf6f9ac1a41b2329ce366710e604e34e

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

PR: https://git.openjdk.org/jdk21u-dev/pull/914


More information about the jdk-updates-dev mailing list