RFR: 8360520: RISC-V: C1: Fix primitive array clone intrinsic regression after JDK-8333154 [v5]

Galder Zamarreño galder at openjdk.org
Tue Jul 22 15:37:09 UTC 2025


On Tue, 22 Jul 2025 14:32:24 GMT, Feilong Jiang <fjiang at openjdk.org> wrote:

>> Hmmm, I don't think I like that name. It's too specific on the flags but does not convey what it's used for. The aim of `flag=0` was to avoid instantiation of array copy stubs, so maybe the name could be `init_flags_for_copy_stubs`? It could be prepended with a `get_` if needs be to avoid confusion.
>
> `init_flags_for_copy_stubs` appears to be misleading, as it suggests we want to generate array copy stubs for those flags.
> How about `get_necessary_copy_flags`? We can add other flags if needed in the future.

I'm unsure about the use of the word `necessary`. What about `get_initial_copy_flags`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25976#discussion_r2222930750


More information about the hotspot-compiler-dev mailing list