Integrated: 8293035: Cleanup MacroAssembler::movoop code patching logic aarch64 riscv

Axel Boldt-Christmas aboldtch at openjdk.org
Wed Aug 31 07:46:16 UTC 2022


On Mon, 29 Aug 2022 15:16:23 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> Cleanup MacroAssembler::movoop logic with respect to code patching on aarch64 and riscv.
> 
> Remove the `bool immediate` argument as it is only used with the value `true` in `void MacroAssembler::movoop(Register dst, jobject obj, bool immediate)`
> 
> Make comments and logic more clear and easier to read.
> 
> Testing: osx, linux aarch64 tier 1-3

This pull request has now been integrated.

Changeset: 9e3176bd
Author:    Axel Boldt-Christmas <aboldtch at openjdk.org>
Committer: Stefan Karlsson <stefank at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9e3176bd09c7210ccb125491abb6a6a4d852b6c6
Stats:     47 lines in 13 files changed: 12 ins; 14 del; 21 mod

8293035: Cleanup MacroAssembler::movoop code patching logic aarch64 riscv

Reviewed-by: eosterlund, fyang

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

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


More information about the hotspot-dev mailing list