RFR: 8357086: os::xxx functions returning memory size should return size_t [v24]

Anton Artemov duke at openjdk.org
Thu Aug 21 08:48:28 UTC 2025


On Wed, 20 Aug 2025 15:41:37 GMT, Joel Sikström <jsikstro at openjdk.org> wrote:

>> src/hotspot/share/runtime/os.hpp line 343:
>> 
>>> 341: 
>>> 342:   static size_t physical_memory();
>>> 343:   static bool has_allocatable_memory_limit(size_t* limit);
>> 
>> Future cleanup: for consistency should this now also take a reference?
>
> This is no longer relevant as it has been re-worked in [JDK-8364248](https://bugs.openjdk.org/browse/JDK-8364248) to not have an out-parameter and renamed to `static size_t commit_memory_limit()`.

Removed in the latest commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25450#discussion_r2290346842


More information about the hotspot-dev mailing list