Integrated: 8318455: Fix the compiler/sharedstubs/SharedTrampolineTest.java and SharedStubToInterpTest.java

Evgeny Nikitin enikitin at openjdk.org
Tue Oct 24 09:19:52 UTC 2023


On Thu, 19 Oct 2023 20:23:18 GMT, Evgeny Nikitin <enikitin at openjdk.org> wrote:

> A continuation of #16258.
> 
> Said tests try to control the compilers used via arguments passed to subprocesses' they create. The [JDK-8314838](https://bugs.openjdk.org/browse/JDK-8314838) added support for additional forwarding of the JTReg javaoptions to the subprocesses, but broke the original arguments passing. They are now passed to the enclosing driver and simply ignored by ProcessTools.createTestJvm. As a result, proper compiler may be not chosen on subprocess start, etc. This PR fixes that.
> 
> Testing: both tests run in CI, with additional arguments (tests were ignored, as expected) and without (passed).

This pull request has now been integrated.

Changeset: 4a16d111
Author:    Evgeny Nikitin <enikitin at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4a16d111b16e556f904fb80075631b8439a9482e
Stats:     9 lines in 3 files changed: 3 ins; 2 del; 4 mod

8318455: Fix the compiler/sharedstubs/SharedTrampolineTest.java and SharedStubToInterpTest.java

Reviewed-by: thartmann, shade, eastigeevich

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

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


More information about the hotspot-compiler-dev mailing list