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