On Tue, 21 Jul 2026 17:02:53 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).
>
> Serguei Spitsyn has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   review: some minor refactoring of the intro on value objects

src/hotspot/share/prims/jvmti.xml line 936:

> 934:     <p/>
> 935:     <internallink id="objecTags">Object tags</internallink> are 
> <code>jlong</code> values
> 936:     that a JVMTI agent can associate with an object.

Suggestion:

    that a <jvmti/> agent can associate with an object.

src/hotspot/share/prims/jvmti.xml line 939:

> 937:     <jvmti/> allows a value object to be tagged despite the object 
> having no identity.
> 938:     Consequently, the tag is a <code>jlong</code> value associated with 
> the object's
> 939:     field values rather than its identity.

Suggestion:

    Value objects can be tagged by <jvmti/> but the tag is associated with the 
object's
    field values, as it has no identity.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2659#discussion_r3626144868
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2659#discussion_r3626210031

Reply via email to