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

Coleen Phillimore coleenp at openjdk.java.net
Fri Jul 9 19:07:57 UTC 2021


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

>> 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.

I filed JDK-8270179 for the follow-on renaming.  It's almost trivial but not quite.

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

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


More information about the hotspot-dev mailing list