RFR: 8329331: Intrinsify Unsafe::setMemory [v11]
Sandhya Viswanathan
sviswanathan at openjdk.org
Thu Apr 11 23:34:43 UTC 2024
On Thu, 11 Apr 2024 20:58:00 GMT, Scott Gibbons <sgibbons at openjdk.org> wrote:
>> src/hotspot/cpu/x86/stubGenerator_x86_64_arraycopy.cpp line 735:
>>
>>> 733:
>>> 734: if (MaxVectorSize == 64) {
>>> 735: UnsafeCopyMemoryMark ucmm(this, !is_oop && !aligned, false, ucme_exit_pc);
>>
>> This is not related to Unsafe::setMemory?
>
> No. Reviewing the code I saw this as a potential error, as `arraycopy_avx3_large` could cause a SIGBUS which wouldn't be caught. It conforms to the other instances of copy in the code. I think it was missed by the original developer.
Would be good to do it in a separate PR then.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18555#discussion_r1561801825
More information about the core-libs-dev
mailing list