RFR: 8364120: RISC-V: unify the usage of MacroAssembler::instruction_size
Fei Yang
fyang at openjdk.org
Mon Jul 28 03:59:00 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
Looks fine. Thanks.
-------------
Marked as reviewed by fyang (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26481#pullrequestreview-3060245312
More information about the hotspot-dev
mailing list