On Wed, 29 Jul 2026 12:31:32 GMT, Christian Hagedorn <[email protected]> wrote:
>> Instead of problemlisting the entire test, we should disable >> TestNullableInlineTypes::test103() in order to reduce the noise in the CI in >> tier6. >> >> Testing `TestNullableInlineTypes` in all possible flag combos (also includes >> more configs than normally seen in the CI): >> - tier1-6 >> - hs-comp-stress + valhalla-comp-stress >> >> Only found a different crash which should be fixed by >> [JDK-8388435](https://bugs.openjdk.org/browse/JDK-8388435) (currently open >> mainline issue). This failure was with a flag combination that we do not see >> in standard CI runs (`-DIgnoreCompilerControls=true -XX:CompileThreshold=100 >> -DVerifyIR=false`). >> >> Note that JDK-8389322 will be fixed after Valhalla integration: >> https://github.com/openjdk/valhalla/pull/2678 >> >> Thanks, >> Christian >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Christian Hagedorn has updated the pull request incrementally with one > additional commit since the last revision: > > Apply suggestion from @TobiHartmann > > Co-authored-by: Tobias Hartmann <[email protected]> Looks good and trivial. test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestNullableInlineTypes.java line 3032: > 3030: Asserts.assertEQ(test103(), null); > 3031: } > 3032: */ Suggestion: */ ------------- Marked as reviewed by thartmann (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2679#pullrequestreview-4808125666 PR Review Comment: https://git.openjdk.org/valhalla/pull/2679#discussion_r3674197250
