RFR: 8276472: align_metadata_size is a nop

David Holmes dholmes at openjdk.java.net
Thu Jan 20 22:11:48 UTC 2022


On Wed, 19 Jan 2022 23:20:37 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> I just added a comment to this and took out the align_up call.  I like align_metadata_size where it is and changing to WordSize doesn't add more safety imo.  Also found unused function while looking for units of metadata size.
> Tested with tier1 on Oracle supported platforms.

Okay. I'm not really sure why we need to make any changes in that case. If the incoming alignment of values were to change then you would want to keep the `align_up` call wouldn't you?

At a minimum should we not have an assertion (static assert?) that the incoming values are actually aligned as expected?

Thanks,
David

Also the issue title is misleading to me as it suggests that being a nop is a problem and that it will be fixed, but it is still a nop after this change.

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

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


More information about the hotspot-dev mailing list