RFR: 8344014: Simplify TracePhase constructor

Thomas Stuefe stuefe at openjdk.org
Wed Nov 13 10:27:37 UTC 2024


On Tue, 12 Nov 2024 20:55:51 GMT, Dean Long <dlong at openjdk.org> wrote:

> This looks OK, but doesn't seem strictly necessary for JDK-8344009. We could get the PhaseTraceId from `accumulator - &Phase::timers[0]`.

I thought so too, but it seemed a bit ugly and brittle, and the simplification seemed worth it anyhow.

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

PR Comment: https://git.openjdk.org/jdk/pull/22029#issuecomment-2473105069


More information about the hotspot-compiler-dev mailing list