On Fri, 28 Feb 2025 10:38:28 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> 
wrote:

>> Chen Liang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   noStride -> constantOffset, optimize VH classes to have only 2 instead of 
>> 3 classes for each type
>
> src/java.base/share/classes/java/lang/invoke/X-VarHandleSegmentView.java.template
>  line 120:
> 
>> 118: 
>> 119: // This class must be accessed through non-aligned 
>> VarHandleSegmentAs$Type$s
>> 120: final class VarHandleSegmentAs$Type$sAligned extends 
>> VarHandleSegmentAs$Type$s {
> 
> I kind of liked the nested classes -- why the change?

Nesting means we need to define 3 classes: AsXxx, Unaligned, Aligned. I wish to 
reduce class loading by merging AsXxx and Unaligned. The Aligned still works as 
if it is nested, but I put it out to make the indentation beautiful for boolean 
and byte.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23720#discussion_r1975417166

Reply via email to