RFR: 8318217: RISC-V: C2 VectorizedHashCode [v2]

Hamlin Li mli at openjdk.org
Wed Nov 29 18:00:11 UTC 2023


On Fri, 24 Nov 2023 18:56:21 GMT, Hamlin Li <mli at openjdk.org> wrote:

>>> BTW, can you add some comments about what java method or bytecode this intrinsic is for?
>> Done.
>
> Hmm, addition of TEMP_DEF result makes the bencmark results even worse tha without intrinsic (I haven't look at the generated assembler though).
> 
> This seems bit confusing to me.

If we use concrete register for `result`, then there is no need to label it as `TEMP_DEF ` either.

So, in summary, let's go with the concrete registers solution if no one has other opinion.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16629#discussion_r1409673888


More information about the hotspot-dev mailing list