On Mon, 15 Jul 2024 14:09:22 GMT, Chen Liang <li...@openjdk.org> wrote:

>> The `@Stable` on the `index` field is incorrect, as stable only avoids 
>> inlining `0`. Solution is to use bit flip on the actual index (and rename 
>> the field to `flippedIndex`), so we use -1 to -256 (mapping to 0 to 255) and 
>> 0 the default value is used as an unset indicator.
>
> Chen Liang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   We have sufficient space in short, use +1 offset

Now I have switched to +1 offset. Looks cleaner?

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

PR Comment: https://git.openjdk.org/jdk/pull/20178#issuecomment-2228713539

Reply via email to