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