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