RFR: 8314838: 3 compiler tests ignore vm flags [v2]

Evgeny Nikitin enikitin at openjdk.org
Thu Sep 7 18:49:15 UTC 2023


> 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.

Evgeny Nikitin has updated the pull request incrementally with one additional commit since the last revision:

  Use vm.simpleArch instead of the os.arch

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15609/files
  - new: https://git.openjdk.org/jdk/pull/15609/files/cd5706e5..83e59965

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15609&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15609&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/15609.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15609/head:pull/15609

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


More information about the hotspot-compiler-dev mailing list