[lworld] RFR: 8378086: [lworld] duplication between ObjArrayKlass and refined classes [v2]
Stefan Karlsson
stefank at openjdk.org
Wed Feb 18 12:52:35 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 incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 23 additional commits since the last revision:
- 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
- Fix more use sites and some TRAPS returns
- ... and 13 more: https://git.openjdk.org/valhalla/compare/d8ceca56...afa01e35
-------------
Changes:
- all: https://git.openjdk.org/valhalla/pull/2117/files
- new: https://git.openjdk.org/valhalla/pull/2117/files/afa01e35..afa01e35
Webrevs:
- full: https://webrevs.openjdk.org/?repo=valhalla&pr=2117&range=01
- incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2117&range=00-01
Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 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