Integrated: 8314838: 3 compiler tests ignore vm flags

Evgeny Nikitin enikitin at openjdk.org
Fri Sep 8 14:11:51 UTC 2023


On Thu, 7 Sep 2023 07:41:44 GMT, Evgeny Nikitin <enikitin at openjdk.org> wrote:

> For the compiler tests in question this PR:
> 
> - replaces all calls to `ProcessTools.createJavaProcessBuilder` with another method, the `createTestJvm`, which prepends given arguments with what was passed to the enclosing test. The change therefore...
> - makes sure the external arguments aren't ignored and are used for creating the inner JVM;
> - adds `@requires` guards in some places - where the test explicitly controls the Tiered Compilation, for example.
> 
> Testing: a local and a CI run, no failures registered.

This pull request has now been integrated.

Changeset: ab6a87e6
Author:    Evgeny Nikitin <enikitin at openjdk.org>
Committer: Evgeny Astigeevich <eastigeevich at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/ab6a87e6705ef7d431ac2955e8501426344a4e1c
Stats:     40 lines in 4 files changed: 9 ins; 14 del; 17 mod

8314838: 3 compiler tests ignore vm flags

Reviewed-by: eastigeevich, kvn, lmesnik

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

PR: https://git.openjdk.org/jdk/pull/15609


More information about the hotspot-compiler-dev mailing list