On Mon, 30 Mar 2026 14:09:22 GMT, Chen Liang <[email protected]> wrote:
>> Paul Hübner has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Move autobox check to ciField.
>
> src/java.base/share/classes/java/lang/Byte.java line 182:
>
>> 180: if (ByteCache.isEnabled()) {
>> 181: final int offset = 128;
>> 182: return ByteCache.cache[(int)b + offset];
>
> I think the introduction of `XxxCache.isEnabled` and all the changes to
> `valueOf` methods are redundant.
Thanks for the input. Axel raised a point regarding this in
https://github.com/openjdk/valhalla/pull/2249#discussion_r3009784200
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2249#discussion_r3010107310
