RFR: 8304824: NMT should not use ThreadCritical [v9]

Johan Sjölen jsjolen at openjdk.org
Mon Nov 4 07:56:38 UTC 2024


On Fri, 1 Nov 2024 21:56:36 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

>This include is not needed because there are no uses that require the definition of Thread.

Right, seems like the forward declaration used to be provided by `memory/allocation.hpp`. Let's get rid of the include and use a forward declaration of our own instead.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20852#discussion_r1827311285


More information about the serviceability-dev mailing list