RFR: 8332042: Move MEMFLAGS to its own include file [v2]
Johan Sjölen
jsjolen at openjdk.org
Mon May 13 06:26:04 UTC 2024
On Mon, 13 May 2024 04:55:24 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> MEMFLAGS, as well as its enum constants, should live in its own include.
>>
>> The constants are used throughout the code base, often without needing the allocation APIs exposed through allocation.hpp.
>>
>> The MEMFLAGS enum def is often needed within NMT itself, again often without needing allocation.hpp.
>>
>> ---
>>
>> This patch moves the enum to its new file.
>>
>> It fixes those `allocation.hpp` includes that where only needed to get MEMFLAGS. It does not fix other includes.
>>
>> For backward compatibility, until we straightened out the dependencies (e.g., fixing all places where we rely on indirect includes), I added memflags.hpp to allocation.hpp.
>>
>> I tested (built) on:
>> - MacOS aarch64, no precompiled headers, fastdebug
>> - Linux x64, no precompiled headers, fastdebug, release, fastdebug crossbuild to aarch64, fastdebug minimal
>
> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>
> Update mallocLimit.hpp
Seems reasonable and LGTM, thanks.
-------------
Marked as reviewed by jsjolen (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19172#pullrequestreview-2051788348
More information about the serviceability-dev
mailing list