> 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
