On Wed, 1 Jul 2026 20:42:52 GMT, Serguei Spitsyn <[email protected]> wrote:

>> This update is for Valhalla pre-integration. It introduces new JVMTI 
>> capability `can_support_value_objects`.
>> The variable `_can_support_value_objects_count` is introduced for 
>> optimization. It follows the pattern of the `can_support_virtual_threads` 
>> capability.
>> 
>> Additionally, this update includes test fixes:
>>  - 
>> test/hotspot/jtreg/serviceability/jvmti/HeapMonitor/libHeapMonitorTest.cpp:
>>    - Removed the `if (!jni->HasIdentity(object))` check  because it is not 
>> needed anymore as the JVMTI capability `can_support_value_objects` is no 
>> acquired by the test
>>  
>>  - These two tests are updated to provide both positive and negative 
>> coverage for new capability:
>>     test/hotspot/jtreg/serviceability/jvmti/valhalla/VMObjectAllocValue
>>     test/hotspot/jtreg/serviceability/jvmti/valhalla/SampledObjectAllocValue
>> 
>> Testing:
>> - Ran updated tests locally
>> - Submitted mach5 tiers 1-6
>> 
>> ---------
>> - [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: add commas in spec updates

Marked as reviewed by cjplummer (Committer).

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

PR Review: 
https://git.openjdk.org/valhalla/pull/2607#pullrequestreview-4613465325

Reply via email to