On Fri, 30 Jan 2026 23:15:37 GMT, Daniel D. Daugherty <[email protected]> wrote:
>> A trivial fix to clarify/organize Valhalla ProblemLists. > > Daniel D. Daugherty has updated the pull request with a new target base due > to a merge or a rebase. The pull request now contains three commits: > > - Merge lworld > - Merge branch 'lworld' into JDK-8376752 > - 8376752: [lworld] clarify/organize Valhalla ProblemLists Checked each PL file by their mainline line number versus the valhalla line numbers. Let's remove the 3 entries accidentally re-added to the PL. test/hotspot/jtreg/ProblemList-enable-preview.txt line 41: > 39: # Runtime: > 40: runtime/cds/serviceability/ReplaceCriticalClassesForSubgraphs.java > 8372599 generic-all > 41: runtime/cds/appcds/cacheObject/ArchivedIntegerCacheTest.java > 8374959 generic-all These two guys are now fixed. Suggestion: test/hotspot/jtreg/ProblemList.txt line 80: > 78: compiler/c2/aarch64/TestStaticCallStub.java 8359963 > linux-aarch64,macosx-aarch64 > 79: > 80: > ############################################################################# Note: mainline has two lines: compiler/longcountedloops/TestLoopNestTooManyTraps.java 8376591 generic-all just added today in 8376781. test/hotspot/jtreg/ProblemList.txt line 117: > 115: containers/docker/TestJcmd.java 8278102 linux-all > 116: containers/docker/TestMemoryAwareness.java 8303470 linux-all > 117: containers/docker/TestJFREvents.java 8327723 linux-x64 Note: TestMemoryAwareness above was just fixed in mainline. test/hotspot/jtreg/ProblemList.txt line 212: > 210: compiler/whitebox/StressNMethodRelocation.java 8370571 generic-all > 211: compiler/valhalla/inlinetypes/TestTrivialMethods.java 8376815 > generic-all > 212: compiler/c2/cmove/TestScalarConditionalMoveCmpObj.java 8374122 > generic-all Suggestion: Fixed last week. test/jdk/ProblemList.txt line 506: > 504: java/lang/invoke/lambda/LambdaFileEncodingSerialization.java 8249079 > linux-all > 505: java/lang/invoke/RicochetTest.java 8251969 > generic-all > 506: Note: mainline added a line after RecochetTest in 8376509: java/lang/ProcessBuilder/PipelineLeaksFD.java 8375585 generic-all ------------- PR Review: https://git.openjdk.org/valhalla/pull/2007#pullrequestreview-3730873065 Changes requested by liach (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2007#pullrequestreview-3730919524 PR Review Comment: https://git.openjdk.org/valhalla/pull/2007#discussion_r2748337007 PR Review Comment: https://git.openjdk.org/valhalla/pull/2007#discussion_r2748344191 PR Review Comment: https://git.openjdk.org/valhalla/pull/2007#discussion_r2748346315 PR Review Comment: https://git.openjdk.org/valhalla/pull/2007#discussion_r2748350382 PR Review Comment: https://git.openjdk.org/valhalla/pull/2007#discussion_r2748361696
