[lworld] RFR: 8378086: [lworld] duplication between ObjArrayKlass and refined classes [v3]
Stefan Karlsson
stefank at openjdk.org
Wed Feb 18 13:48:06 UTC 2026
> I have a bunch of patches to remove some code duplication, redundant setting of fields, unneeded dispatch code.
>
> I've tested this with tier1-3.
>
> I'm opening this up as one PR, depending on #2033, but it could be split up into multiple dependent PRs. It could be good to look at each commit individually when making a first pass of over this PR.
>
> I'll add a guide to the individual commits as a comment after the summary.
Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 24 commits:
- Merge remote-tracking branch 'valhalla/lworld' into 8378086_obj_array_klass_code_deduplication
- Add temporary assert
- Restore ObjArrayKlass private
- Remove create_element_klass_array_name duplication
- Remove ObjArrayKlass::oop_verify_on implementation
- Remove ObjArrayKlass::oop_size
- Remove ObjArrayKlass::copy_array
- Fix incorrect reference to RefArrayKlass::
- Deduplicate array klass code
- Fixes from Stefan's comments
- ... and 14 more: https://git.openjdk.org/valhalla/compare/646c2f93...24bd1bb1
-------------
Changes: https://git.openjdk.org/valhalla/pull/2117/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2117&range=02
Stats: 256 lines in 12 files changed: 48 ins; 164 del; 44 mod
Patch: https://git.openjdk.org/valhalla/pull/2117.diff
Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2117/head:pull/2117
PR: https://git.openjdk.org/valhalla/pull/2117
More information about the valhalla-dev
mailing list