On Fri, 13 Mar 2026 22:39:12 GMT, Chris Plummer <[email protected]> wrote:

>> test/hotspot/jtreg/serviceability/jvmti/HeapMonitor/libHeapMonitorTest.cpp 
>> line 556:
>> 
>>> 554:       jni->FatalError("Error in event_storage_add: Exception in jni 
>>> NewWeakGlobalRef");
>>> 555:     }
>>> 556: 
>> 
>> I'm thinking if this can be simplified by checking if the object is value 
>> object or not before creation of a week global ref. What was a motivation to 
>> do the way above?
>
> I agree. If it is a value object, then just skip it and return.

fixed.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2230#discussion_r2934399498

Reply via email to