RFR: 8330465: Stable Values and Collections (Internal) [v5]
Per Minborg
pminborg at openjdk.org
Thu May 16 07:14:17 UTC 2024
On Wed, 15 May 2024 16:11:56 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Switch to monomorphic StableValue and use lazy arrays
>
> src/java.base/share/classes/jdk/internal/lang/stable/StableValueImpl.java line 139:
>
>> 137: case NON_NULL: { return valueVolatile(); }
>> 138: case ERROR: { throw StableUtil.error(this); }
>> 139: case DUMMY: { throw shouldNotReachHere(); }
>
> Redundant branch?
The idea here is to have the most likely value in the middle... Not sure if that motivates the added complexity though.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18794#discussion_r1602739126
More information about the hotspot-compiler-dev
mailing list