Integrated: 8297830: aarch64: Make Address a discriminated union internally

Kim Barrett kbarrett at openjdk.org
Thu Dec 1 20:19:31 UTC 2022


On Wed, 30 Nov 2022 11:13:35 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this change to the aarch64 Address class.  It now uses an
> internal union, separating the literal and nonliteral cases.
> 
> This avoids leaving some fields uninitialized or initializing them to dummy
> values.  It also reduces the size of the Address class somewhat, though it's
> unclear whether that makes any noticeable difference.
> 
> Testing:
> mach5 tier1 for linux-aarch64 and macosx-aarch64
> mach5 tier2-5 for linux-aarch64

This pull request has now been integrated.

Changeset: 5a5ced3a
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/5a5ced3a900a81fd0b0757017f4138ce97e2521e
Stats:     205 lines in 3 files changed: 110 ins; 16 del; 79 mod

8297830: aarch64: Make Address a discriminated union internally

Reviewed-by: aph, dcubed

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

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


More information about the hotspot-dev mailing list