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

[Kim Barrett]

On Fri, 9 Jul 2021 01:25:15 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Thanks to @kimbarrett for noticing this. The alignment was changed to 64 bits for 32 bit platforms, but overalign for 64 bits platforms.  I changed this to BytesPerLong to cover both, since the long case is why it was changed on 32 bits in the first place in JDK-4526490.
>> I also removed Amalloc_D since I don't know what D stands for and it's the same as Amalloc_4.  That's not a great name either. I'm open to suggestions!
>> Tested with tier1-3.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Improvements ala Kim

Yuck!  The handling of the replies to "outdated" conversations seems pretty horrible.  In "conversation" mode they remain attached to the original change, while in "files changed" mode they don't show up at all.  Sorry this seems to be kind of a mess.  I also have no idea why these got tagged as "outdated".

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

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