RFR: 8310268: RISC-V: misaligned memory access in String.Compare intrinsic [v3]

Vladimir Kempik vkempik at openjdk.org
Mon Jul 24 13:22:41 UTC 2023


On Mon, 24 Jul 2023 12:45:58 GMT, Vladimir Kempik <vkempik at openjdk.org> wrote:

>> Ah, I see you used `t0` instead. I once considered that, but I would still suggest `x7` which would be safer. `t0` as a scratch register are implicitly clobbered/used by so many assembler functions that it's risky to keep a live value in it across assemblers like `inflate_lo32`. `x7` should be a better candidate here as it is only updated at label `CALCULATE_DIFFERENCE` in function generate_compare_long_string_different_encoding.
>
> Good points

Updated the PR, this register renaming hasn't changed performance on hifive compared to previous version

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14534#discussion_r1272254708


More information about the hotspot-dev mailing list