RFR: 8356172: IdealGraphPrinter doesn't need ThreadCritical

Christian Hagedorn chagedorn at openjdk.org
Tue May 6 08:34:13 UTC 2025


On Mon, 5 May 2025 12:27:22 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> Please review this possibly trivial change.
> Tested with hs-precheckin-comp test list.

I once just moved the code around. But the `ThreadCritical` seems to be here since the initial load. So, I'm not sure if the exact reason is still known. But it sounds reasonable to remove the `ThreadCritical`.

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25035#pullrequestreview-2817361509


More information about the hotspot-compiler-dev mailing list