RFR: 8312132: Add tracking of multiple address spaces in NMT [v74]

Thomas Stuefe stuefe at openjdk.org
Mon May 13 12:54:12 UTC 2024


On Mon, 13 May 2024 11:05:47 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Oh, another thing, maybe rename this to something else. `Metadata` has a clear meaning in hotspot. Maybe something like RegionData?
>
> Hi Thomas, what naming scheme should I use for the empty static RegionData? `EmptyRegionData` or `empty_regiondata`? It can't be `constexpr`, but we can make it `const`.

If its scoped to RegionData, I would just call it "empty". `RegionData::empty` is clear enough. If its global scope, I'd call it empty_regiondata.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18289#discussion_r1598423629


More information about the hotspot-dev mailing list