RFR: 8354244: Use random data in MinMaxRed_Long data arrays
Galder Zamarreño
galder at openjdk.org
Thu Jul 24 06:51:36 UTC 2025
Simplified the data used in the tests added in [JDK-8307513](https://bugs.openjdk.org/browse/JDK-8307513). The data does not need to have a specific shape because this test focuses on verifying the IR when vectorization kicks in, and when it does, the data can just be random. Shaping the data to control branch taken/not-taken paths makes sense when CMov macro expansion kicks in instead of vectorization.
When switching to random data I noticed that the test was randomly failing. This was due to potential overflows that result from takin the min/max and then multiplying it by 11, so I've adjusted that section of the test as well.
I've run the test on both aarch64 and x64 platforms where this test would get vectorized. To verify that I made sure the test passed and verified that the jtr output to make sure the IR conditions were matched.
-------------
Commit messages:
- Simplify test
Changes: https://git.openjdk.org/jdk/pull/26451/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26451&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8354244
Stats: 84 lines in 1 file changed: 11 ins; 62 del; 11 mod
Patch: https://git.openjdk.org/jdk/pull/26451.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26451/head:pull/26451
PR: https://git.openjdk.org/jdk/pull/26451
More information about the hotspot-compiler-dev
mailing list