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