[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