RFR: 8293254: x86_64: Extract arraycopy stub implementations into a separate file [v2]

Vladimir Ivanov vlivanov at openjdk.org
Fri Sep 2 17:12:16 UTC 2022


> Move arraycopy stub implementations from `stubGenerator_x86_64.cpp` into `stubGenerator_x86_64_arraycopy.cpp`.
> 
> Testing: hs-tier1 - hs-tier4
> 
> PS: FTR I left `MacroAssembler::generate_fill_avx3()` intact because it is called from `MacroAssembler::generate_fill()` which is shared between x86_64 and x86_32. Otherwise, I would either need to copy `MacroAssembler::generate_fill()` or put it into a separate file (e.g., `stubGenerator_x86.cpp` or `stubGenerator_x86_arraycopy.cpp`). IMO it doesn't add much value compared to keeping it in `macroAssembler_x86.cpp`.

Vladimir Ivanov 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.

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/10128/files
  - new: https://git.openjdk.org/jdk/pull/10128/files/e518a78d..e518a78d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10128&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10128&range=00-01

  Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/10128.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10128/head:pull/10128

PR: https://git.openjdk.org/jdk/pull/10128


More information about the hotspot-dev mailing list