[lworld] Integrated: 8267818: [lworld] [AArch64] Shenandoah barrier set build warnings and register conflict
Nick Gasson
ngasson at openjdk.java.net
Mon May 31 09:32:32 UTC 2021
On Thu, 27 May 2021 10:26:03 GMT, Nick Gasson <ngasson at openjdk.org> wrote:
> I get this warning when building with Shenandoah enabled on AArch64:
>
> [...]/barrierSetAssembler_aarch64.hpp:47:16: warning: 'virtual void BarrierSetAssembler::store_at(MacroAssembler*, DecoratorSet, BasicType, Address, Register, Register, Register, Register)' was hidden [-Woverloaded-virtual]
>
> BarrierSetAssembler::store_at() gained an extra tmp3 argument. (The
> same fix was already applied on x86.)
>
> ShenahdoahBarrierSetAssembler::load_at() uses rscratch1 and rscratch2
> internally but the interpreter _fast_qgetfield implementation passes
> rscratch1 as a separate temporary via get_default_value_oop() which
> causes an assert_different_registers() failure. Instead just allocate a
> fresh temporary register and pass that.
This pull request has now been integrated.
Changeset: 44cb67ea
Author: Nick Gasson <ngasson at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.java.net/valhalla/commit/44cb67ea3457e7359a3c055a08a4105bbd068564
Stats: 7 lines in 3 files changed: 0 ins; 0 del; 7 mod
8267818: [lworld] [AArch64] Shenandoah barrier set build warnings and register conflict
Reviewed-by: fparain
-------------
PR: https://git.openjdk.java.net/valhalla/pull/430
More information about the valhalla-dev
mailing list