Integrated: 8320370: NMT: Change MallocMemorySnapshot to simplify code.
Johan Sjölen
jsjolen at openjdk.org
Thu Dec 7 16:01:44 UTC 2023
On Sat, 18 Nov 2023 15:48:19 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
> Hi,
>
> `MallocMemorySnapshot` used to be initialized in a quite non-standard way, using global placement new on an array of `size_t` of the size required to fit `MallocMemorySnapshot`. This looks like it was intended to circumvent some `ResourceObj` internals, but I'm unsure of its purpose. This change does what you expect a regular initialization of a global variable to look like.
>
> ~Currently running through tier1.~ GHA passed and so did tier1 on Oracle CI.
This pull request has now been integrated.
Changeset: a7f60164
Author: Johan Sjölen <jsjolen at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/a7f60164063bdf95437326e7550fd65ff91b5566
Stats: 19 lines in 5 files changed: 0 ins; 12 del; 7 mod
8320370: NMT: Change MallocMemorySnapshot to simplify code.
Reviewed-by: stuefe, gziemski, stefank
-------------
PR: https://git.openjdk.org/jdk/pull/16724
More information about the hotspot-runtime-dev
mailing list