RFR: 8323779: Serial: Remove Generation::promotion_attempt_is_safe
Stefan Karlsson
stefank at openjdk.org
Tue Jan 16 10:04:22 UTC 2024
On Tue, 16 Jan 2024 09:16:08 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
> Simple updating a field type to use more concrete type.
src/hotspot/share/gc/serial/defNewGeneration.hpp line 56:
> 54: friend class VMStructs;
> 55:
> 56: TenuredGeneration* _old_gen;
The alignment in this block is weird after this change.
src/hotspot/share/gc/serial/tenuredGeneration.hpp line 163:
> 161: // Promotion of the full amount is not guaranteed but
> 162: // might be attempted in the worst case.
> 163: bool promotion_attempt_is_safe(size_t max_promoted_in_bytes) const;
Could you move this declaration up above the `// Performance Counter Support`, so that it doesn't look like this function blongs to the `// Statistics` section?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17439#discussion_r1453191761
PR Review Comment: https://git.openjdk.org/jdk/pull/17439#discussion_r1453194103
More information about the hotspot-gc-dev
mailing list