RFR: 8224509: Incorrect alignment in CDS related allocation code on 32-bit platforms

Thomas Stuefe stuefe at openjdk.java.net
Fri Oct 2 05:15:05 UTC 2020


On Thu, 1 Oct 2020 22:40:46 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> CDS aligns pointers and sizes to BytesPerWord. On 64-bit, BytesPerWord = 8. On 32-bit, BytesPerWord = 4.
> 
> To make the alignment consistent between 64- and 32-bit platforms, this patch changes the alignment value to 8.
> 
> Testing: tiers 1 - 4
>                built linux-x86-debug.

Just a drive by comment. Otherwise looks good to my non-CDS-eyes.

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

Marked as reviewed by stuefe (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/476


More information about the hotspot-runtime-dev mailing list