RFR: 8302850: Implement C1 clone intrinsic that reuses arraycopy code for primitive arrays [v6]

Tue Mar 12 08:22:14 UTC 2024

On Fri, 8 Mar 2024 01:42:03 GMT, Dean Long <dlong at openjdk.org> wrote:

> Your front-end changes require back-end changes, which are only implemented for x86 and aarch64. So you need a way to disable this for other platforms, or port the fix to all platforms. Minimizing the amount of platform-specific code required would also help.

I'm struggling to understand what it is you think is missing in the PR. I have added the following 2 sections in such a way that they only trigger in x86 and aarch64. See [here](https://github.com/openjdk/jdk/pull/17667/files#diff-737789206706361d06d1f120e10272b62bcfdb556e8e73693f94ec87f2a6b369R238) and [here](https://github.com/openjdk/jdk/pull/17667/files#diff-e6f3ae4492965efd0d73c3f31073ec8b77e020740b009f92312658bac1e5f978R356), and as far as I understand it, that's enough to address your concerns. Please let me know if there is something I might have missed.

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

PR Comment: https://git.openjdk.org/jdk/pull/17667#issuecomment-1991014592