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

Galder Zamarreño galder at openjdk.org
Mon Apr 15 15:49:18 UTC 2024


On Thu, 11 Apr 2024 17:41:44 GMT, Dean Long <dlong at openjdk.org> wrote:

>>> Sure, feel free to improve it.
>> 
>> I've just commented in https://github.com/openjdk/jdk/pull/18642/files#r1560900520, what do you think?
>> 
>>> I fixed the issue I had, so it should be good to test out now. Thanks.
>> 
>> Ok, so how would we integrate your changes? Do I just merge the commits in your https://github.com/dean-long/jdk/tree/pr/17667 branch?
>
>> Ok, so how would we integrate your changes? Do I just merge the commits in your https://github.com/dean-long/jdk/tree/pr/17667 branch?
> 
> Yes, I think that should work.  If that doesn't add me as a contributor, then I can be added manually.

@dean-long I've added your commits on top of the PR, and added an additional commit to include https://github.com/openjdk/jdk/pull/18642#discussion_r1560900520. I've run compiler tests limiting to C1 on x86_64 and things looked good. I didn't test other architectures because the additional changes were limited to the shared code.

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

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


More information about the hotspot-compiler-dev mailing list