RFR: 8285980: Several tests in compiler/c2/irTests miss @requires vm.compiler2.enabled
Jie Fu
jiefu at openjdk.java.net
Mon May 2 02:00:12 UTC 2022
Hi all,
Several tests in compiler/c2/irTests fail if C2 is not available.
The following 7 tests use C2-only VM falgs.
compiler/c2/irTests/TestSkeletonPredicates.java
compiler/c2/irTests/TestFewIterationsCountedLoop.java
compiler/c2/irTests/TestDuplicateBackedge.java
compiler/c2/irTests/TestStripMiningDropsSafepoint.java
compiler/c2/irTests/TestCountedLoopSafepoint.java
compiler/c2/irTests/TestLongRangeChecks.java
compiler/c2/irTests/TestSuperwordFailsUnrolling.java
The following two tests assert that C2 must be available.
compiler/c2/irTests/TestIRLShiftIdeal_XPlusX_LShiftC.java
compiler/c2/irTests/TestIRAddIdealNotXPlusC.java
It would be better to add `@requires vm.compiler2.enabled` to these tests.
Thanks.
Best regards,
Jie
-------------
Commit messages:
- 8285980: Several tests in compiler/c2/irTests miss @requires vm.compiler2.enabled
Changes: https://git.openjdk.java.net/jdk/pull/8495/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8495&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8285980
Stats: 9 lines in 9 files changed: 9 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/8495.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/8495/head:pull/8495
PR: https://git.openjdk.java.net/jdk/pull/8495
More information about the hotspot-compiler-dev
mailing list