RFR: 8349764: RISC-V: C1: Improve Class.isInstance intrinsic [v2]

[Gui Cao]

On Thu, 13 Feb 2025 09:06:36 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Yes, I think we can put a direct return here.
>
> Yes, seems better.
> I was thinking to move bind(fail) here, but seems current lookup_secondary_supers_table_var only accept a success label.

> Hmm, I think this jump could be saved if we put a direct return here instead. Like:
> 
> ```
>         __ beq(klass, x17, success);
>         __ mv(result, 0);
>         __ ret();
> ```
> 
> What do you think? @Hamlin-Li @zifeihan

Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23551#discussion_r1954301888