On Fri, 17 Jul 2026 02:34:27 GMT, David Holmes <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: reconsidered again; small tweak for onject tagging spec > > src/hotspot/share/prims/jvmti.xml line 14232: > >> 14230: When preview features are enabled, the Object Free event is >> only sent for tagged >> 14231: <externallink id="jni/functions.html#hasidentity">identity >> objects</externallink>. >> 14232: <b>The event is not sent for tagged value objects.</b> > > Suggestion: > > <b>The event is not sent for tagged value objects because they are kept > alive while tagged.</b> > > To be consistent with earlier change. We have discussed with you that this is confusing as the lifecycle of value objects is not defined. My move is in opposite direction. It is to remove this: "because they are kept alive while tagged." ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2634#discussion_r3600779223
