RFR: 8356173: Remove ThreadCritical [v2]

Coleen Phillimore coleenp at openjdk.org
Fri May 9 18:55:54 UTC 2025


On Fri, 9 May 2025 14:56:42 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> You can use `Deferred<T>`, but as `PlatformMutex` is at "the bottom" of the initialization order it's not necessary, as Thomas notes. I'd be happy to see it use `Deferred<T>`, as it can help with debugging if PlatformMutex ever changes.
>
> Not really crazy about the Deferred<> name and that change is still under consideration.
> edit: I didn't realize it got checked in.

Deferred<> doesn't help me because it still has to call the initialize function, which is the thing we don't want to have to call.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25072#discussion_r2082343792


More information about the hotspot-dev mailing list