Integrated: 8332899: RISC-V: add comment and make the code more readable (if possible) in MacroAssembler::movptr

Hamlin Li mli at openjdk.org
Fri May 31 15:04:39 UTC 2024


On Tue, 28 May 2024 15:36:00 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review the patch?
> As discussed, https://github.com/openjdk/jdk/pull/19246#discussion_r1613279908, it's worth to make the code more readable.
> For movptr1, add some comments to help understand the tricky part.
> For movptr2, it uses the similar (tricky) way as movptr1, so I align the code implementation with movptr1, and try to make it more straightforward.
> I tried it, hope it's better.
> Thanks.

This pull request has now been integrated.

Changeset: 914423e3
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/914423e3b7162ad934fa4edc46ee37e0f401d27b
Stats:     32 lines in 1 file changed: 25 ins; 3 del; 4 mod

8332899: RISC-V: add comment and make the code more readable (if possible) in MacroAssembler::movptr

Reviewed-by: rehn, fyang

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

PR: https://git.openjdk.org/jdk/pull/19431


More information about the hotspot-dev mailing list