RFR: 8346916: [REDO] align_up has potential overflow [v3]

Casper Norrbin cnorrbin at openjdk.org
Mon Mar 3 16:57:13 UTC 2025


On Mon, 3 Mar 2025 13:52:06 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Changed it to this now
>
> Thank you; my suggestion was insufficient... Need to have an early-return when `uncommitted_size` is 0 at the beginning of `PSOldGen::expand`. After this, I wonder if `align_up_or_min` is truly warranted.

Added the extra check, tests should now work as expected!

I agree with your thoughts on `align_up_or_min`. Since it's no longer used, I went ahead and removed it. Now we. only have `align_up_or_null` left for pointer types.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23711#discussion_r1977853877


More information about the hotspot-dev mailing list