On Tue, 21 Jul 2026 08:35:17 GMT, Serguei Spitsyn <[email protected]> wrote:
> This is a minor tweak in the JVMTI intro section with general clarification > for value objects suggested by @AlanBateman. > This is for Valhalla pre-integration. > > Testing: N/A > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). src/hotspot/share/prims/jvmti.xml line 941: > 939: <p/> > 940: In summary, tags for value objects use value equality semantics, and > 941: the <eventlink id="ObjectFree"/> event is not sent for tagged value > objects. I think it would be better to have just one paragraph for this topic in the identityAndValueObjects section. How about just adding "The ObjectFree event is not sent for tagged value objects" to the first paragraph, drop the "In summary ..." line, and put the paragraph break before the sentence on VMObjectAlloc/SampledObjectAlloc as they are different feature. It would also be useful if "Object tags" could link to the objectTags section. Also the first use of jlong can go into a code tag so it's mono space like the other usages. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2659#discussion_r3622638902
