[lworld] RFR: 8376461: [lworld] serviceability/jvmti/valhalla/HeapDump/HeapDump.java fails More than 1 instances of 'TestClass' found

[Serguei Spitsyn]

On Fri, 13 Feb 2026 21:20:23 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

> TestClass is a container class which contains a number of value object, main test process and target VM create TestClass object, so the test can compare the object and its representation in the target VM heap dump.
> After JDK-8376135 `new TestClass()` may cause unreachable TestClass instance is still in the heap, so it appears in the heap dump too.
> The simplest solution is to make TestClass non-value class (actual tested value classes are nested classes of TestClass)

Looks reasonable.

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

Marked as reviewed by sspitsyn (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/2109#pullrequestreview-3823533146