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