Integrated: 8285980: Several tests in compiler/c2/irTests miss @requires vm.compiler2.enabled

[Jie Fu]

On Mon, 2 May 2022 01:53:57 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 1f9f8738
Author:    Jie Fu <jiefu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/1f9f8738f344ecbc0270608ee84eb92138f349a2
Stats:     9 lines in 9 files changed: 9 ins; 0 del; 0 mod

8285980: Several tests in compiler/c2/irTests miss @requires vm.compiler2.enabled

Reviewed-by: thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/8495