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