RFR: 8315969: compiler/rangechecks/TestRangeCheckHoistingScaledIV.java: make flagless [v2]

Evgeny Nikitin enikitin at openjdk.org
Tue Nov 21 07:13:39 UTC 2023


> A recent improvement ([JDK-8314838](https://bugs.openjdk.org/browse/JDK-8314838)) changed dependant VM creation in that test - we now forward given arguments via 'ProcessTools.createTestJvm()'. It has been revealed, however, that the test doesn't work well with '-Xcomp'. A bug ([JDK-8316124](https://bugs.openjdk.org/browse/JDK-8316124)) has been filed.
> 
> I suggest to temporarily make test 'vm.flagless' to allow for running the test in CI. As soon as [JDK-8316124](https://bugs.openjdk.org/browse/JDK-8316124) is fixed, we'll remove that limitation.

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 two additional commits since the last revision:

 - Merge remote-tracking branch 'upstream/master' into JDK-8315969/make_TestRangeCheckHoistingScaledIV_vm_flagless/pub
 - 8315969: compiler/rangechecks/TestRangeCheckHoistingScaledIV.java: make flagless

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/16746/files
  - new: https://git.openjdk.org/jdk/pull/16746/files/532347a0..a6f7aa62

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

  Stats: 3943 lines in 14 files changed: 2460 ins; 274 del; 1209 mod
  Patch: https://git.openjdk.org/jdk/pull/16746.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16746/head:pull/16746

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


More information about the hotspot-compiler-dev mailing list