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