RFR: 8333729: C2 SuperWord: remove some @requires usages in test/hotspot/jtreg/compiler/loopopts/superword
Emanuel Peter
epeter at openjdk.org
Wed Jun 12 08:33:35 UTC 2024
I'm removing as many `@requires` statements as possible. Generally, they just limit what tests can be executed where, and in most cases such limitations do not make much sense.
For example, tests should not be limited to `@requires vm.compiler2.enabled`, i.e. only run if C2 is available - we can also run the tests if GraalVM is available.
We should also not limit tests to `@requires vm.bits == "64"`, but rather limit the `@IR` rules with `applyIfPlatform = {"64-bit", "true"}`.
Similarly, we should not limit the platform for a run statement, but do that per `@IR` rule if necessary.
-------------
Commit messages:
- last file
- Merge branch 'master' into JDK-8333729-superword-requires
- JDK-8333729
Changes: https://git.openjdk.org/jdk/pull/19581/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19581&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8333729
Stats: 57 lines in 20 files changed: 20 ins; 32 del; 5 mod
Patch: https://git.openjdk.org/jdk/pull/19581.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19581/head:pull/19581
PR: https://git.openjdk.org/jdk/pull/19581
More information about the hotspot-compiler-dev
mailing list