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