RFR: Make the compiler/sharedstubs/SharedTrampolineTest.java require C2 explicitly [v2]
Evgeny Nikitin
enikitin at openjdk.org
Thu Oct 19 11:07:52 UTC 2023
> The test seem to only work with C2. A previous change to the test, the JDK-8314838, allowed for passing JVM arguments to the child process. However, if those parameters change the compiler used, the test fails.
>
> This PR makes this requirement (to have C2 as compiler) explicit.
>
> Testing: runs with '-XX:TieredStopAtLevel=3' with old version (failed), fixed version (skipped) and with new version but no additional arguments (passed).
Evgeny Nikitin has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Revert "8318471: ProblemList compiler/sharedstubs/SharedTrampolineTest.java"
This reverts commit e25a49a993f270c33f7929e629fb3075a11fdec9.
- Merge remote-tracking branch 'origin/master' into JDK-8318455/shared_trampoline_test_requires_c2/pub
- 8318455: Make the compiler/sharedstubs/SharedTrampolineTest.java require C2 explicitly
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16258/files
- new: https://git.openjdk.org/jdk/pull/16258/files/f0b9a894..8b79797e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16258&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16258&range=00-01
Stats: 15531 lines in 666 files changed: 9891 ins; 2929 del; 2711 mod
Patch: https://git.openjdk.org/jdk/pull/16258.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16258/head:pull/16258
PR: https://git.openjdk.org/jdk/pull/16258
More information about the hotspot-compiler-dev
mailing list