On Mon, 15 Jul 2024 15:57:30 GMT, Chen Liang <li...@openjdk.org> wrote:
>> The `@Stable` on the `index` field is incorrect, as stable only avoids >> inlining `0`. On a strategic view, this index field should just become final >> so that `Name` becomes eligible for value class migration once valhalla >> comes. This patch makes the `index` field final and updates the usages >> correspondingly. > > Chen Liang has updated the pull request incrementally with one additional > commit since the last revision: > > Encapsulate offsetIndex, share computation result I have reimplemented this patch to implement the "make `index` field final" approach, which is much cleaner and friendlier to valhalla. tier 1-3 tests pass. ------------- PR Comment: https://git.openjdk.org/jdk/pull/20178#issuecomment-2229569211
