Integrated: 8364120: RISC-V: unify the usage of MacroAssembler::instruction_size

Hamlin Li mli at openjdk.org
Mon Jul 28 13:03:04 UTC 2025


On Fri, 25 Jul 2025 14:00:54 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this simple cleanup?
> 
> in MacroAssembler, some magic number 4 is used for MacroAssembler::instruction_size. We should not use magic number if possible.
> This pr also replaces all the usage of  `instruction_size` with `MacroAssembler::instruction_size`.
> 
> Thanks

This pull request has now been integrated.

Changeset: 7bf4c608
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7bf4c608e7b23398ea89cf0a592e9554af98144e
Stats:     38 lines in 2 files changed: 0 ins; 0 del; 38 mod

8364120: RISC-V: unify the usage of MacroAssembler::instruction_size

Reviewed-by: fyang

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

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


More information about the hotspot-dev mailing list