RFR: 8253779: Amalloc may be wasting space by overaligning [v2]

Coleen Phillimore coleenp at openjdk.java.net
Fri Jul 9 14:30:55 UTC 2021


On Fri, 9 Jul 2021 14:27:21 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I agree. I feel like this should really be the default "Amalloc" (pointer-sized alignment) and Amalloc should really be specific, e.g. "Amalloc64".
>
> For some reason, the new me doesn't want to change all of these calls, but there are _only_ 45 Amalloc and now 38 Amalloc_4s.
> I agree with both of you though.  Amalloc should be the default pointer sized alignment, and existing Amalloc should be jlong aligned.  So Amalloc_4 => Amalloc and Amalloc => Amalloc64.  I suspect the Amalloc64s could really be Amalloc but that only matters on 32 bit platforms which we don't support.  They can be changed by supporters of 32 bit platforms though.

It seems like this can do that renaming to get this done.  The reason I worked on this enhancement is to get it out of the backlog, so I don't want to add another fairly simple RFE to the backlog.

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

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


More information about the hotspot-dev mailing list