> 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
