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