On Sat, 18 Jul 2026 18:35:06 GMT, Chen Liang <[email protected]> wrote:

>> @jddarcy noted [JDK-8387875](https://bugs.openjdk.org/browse/JDK-8387875) 
>> did not provide links from wrapper class cache mechanism declarations to the 
>> overview documentation. We can add comments and also enhance that section in 
>> the design doc.
>> 
>> Also the value-class-preview.html is a bit out of sync.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Chen Liang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Consistently use non-code javac

doc/value-class-preview.md line 112:

> 110: 
> 111: In interpreter or C1 execution in Hotspot, allocations of value objects 
> to the
> 112: heap as a full object with header happens when a value object is:

Suggestion:

heap as a full object with header happen when a value object is:

doc/value-class-preview.md line 115:

> 113: 
> 114: 1. Loaded from a flat storage (field or array)
> 115: 2. Created by a constructor

This is not a complete list. For example, allocations will also happen at C2 to 
C1/interpreter calls and returns if the scalarized calling convention is used.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2655#discussion_r3612179741
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2655#discussion_r3612186126

Reply via email to