RFR: 8351140: RISC-V: Intrinsify Unsafe::setMemory [v14]
Anjian-Wen
duke at openjdk.org
Thu May 22 04:39:43 UTC 2025
On Thu, 22 May 2025 03:03:13 GMT, Anjian-Wen <duke at openjdk.org> wrote:
> > The last change on the code after L_fill_elements make the byte store 1 by 1, it seems that 'true' or 'false' may not affect it when the count is less or equal 7?
>
> Ah, I see.
>
> > I think the above align section is like a fast path which can reduce instruction for the count is large than 7, so we may should check the result when count > 7 ? besides, I'm testing on that right now(delete the align tail part), we can find it out later
>
> Thanks, let's see the test result.
@Hamlin-Li The original branch rarely walks, which is almost never tested, and the performance impact of the ‘fast’ path is relatively small. I think you are right, it is suitable to be deleted
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23890#issuecomment-2899870707
More information about the hotspot-compiler-dev
mailing list