JDK-8314838 : 3 compiler tests ignore vm flags
  • Type: Sub-task
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 22
  • Priority: P5
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2023-08-22
  • Updated: 2024-03-22
  • Resolved: 2023-09-08
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 21 JDK 22
21.0.3Fixed 22 b15Fixed
Related Reports
Relates :  
Relates :  
Relates :  
Relates :  
Description
Tests

compiler/sharedstubs/SharedStubToInterpTest.java
compiler/rangechecks/TestRangeCheckHoistingScaledIV.java
compiler/ciReplay/TestInvalidReplayFile.java

ignore vm flags given for testing.
Comments
Fix request [21u] I backport this to keep the 21u test suite up-to-date. This will simplify future test backports. No risk, only a test change. Clean backport. Tests pass. SAP nightly testing passed.
16-12-2023

A pull request was submitted for review. URL: https://git.openjdk.org/jdk21u-dev/pull/38 Date: 2023-12-15 09:50:44 +0000
15-12-2023

Changeset: ab6a87e6 Author: Evgeny Nikitin <enikitin@openjdk.org> Committer: Evgeny Astigeevich <eastigeevich@openjdk.org> Date: 2023-09-08 14:09:10 +0000 URL: https://git.openjdk.org/jdk/commit/ab6a87e6705ef7d431ac2955e8501426344a4e1c
08-09-2023

A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/15609 Date: 2023-09-07 07:41:44 +0000
07-09-2023

[~thartmann], I'll fix them. How urgent is it?
25-08-2023

Thanks, [~eastigeevich]. Would you have time to fix these tests (any maybe the other ones listed above as well)?
25-08-2023

Hi [~thartmann], I have no objection to use `ProcessTools.createTestJvm` instead of `ProcessTools.createJavaProcessBuilder` or to mark them flagless in: - compiler/sharedstubs/SharedStubToInterpTest.java - compiler/sharedstubs/SharedTrampolineTest.java
24-08-2023

ILW = Tests ignore VM flags (should be marked or pass flags), 3, tests, no workaround = LLH = P5 Tests were added by JDK-8289996, JDK-8280481, JDK-8292561. [~eastigeevich], [~stuefe], [~pli] what do you think?
23-08-2023