[foreign-memaccess+abi] Integrated: Rename max alignment constants for heap segments

Maurizio Cimadamore mcimadamore at openjdk.org
Tue Sep 26 16:17:41 UTC 2023


On Tue, 26 Sep 2023 14:39:42 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> The name of the max alignment constants in `HeapMemorySegmentImpl` is misleading: these constants use numeric values (e.g. `MAX_ALIGN_8`) but in reality the maximum supported alignment can vary depending on the platform (at least for segments wrapping `long[]` and `double[]` arrays).
> 
> This PR introduces a less confusing naming scheme, and a comment to call out the fact that some constants are platform-dependent.

This pull request has now been integrated.

Changeset: 319382bc
Author:    Maurizio Cimadamore <mcimadamore at openjdk.org>
URL:       https://git.openjdk.org/panama-foreign/commit/319382bc25d8656478846d9af68a22b9f7e75696
Stats:     17 lines in 1 file changed: 5 ins; 1 del; 11 mod

Rename max alignment constants for heap segments

Reviewed-by: jvernee

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

PR: https://git.openjdk.org/panama-foreign/pull/897


More information about the panama-dev mailing list