RFR: 8278079: C2: expand_dtrace_alloc_probe doesn't take effect in macro.cpp

Tobias Hartmann thartmann at openjdk.java.net
Fri Dec 3 08:03:16 UTC 2021


On Thu, 2 Dec 2021 15:16:24 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> I think that explains why ciEnv need to cache DTrace flags before compilation, and _dtrace_method_probes/_dtrace_alloc_probes is set enabled if ExtendedDTraceProbes is true, so I think ExtendedDTraceProbes may still be useful, right?

Yes, it makes perfect sense that the flag values are cached but what I'm saying is that with your change, `-XX:+ExtendedDTraceProbes` degrades to an alias for `-XX:+DTraceMethodProbes -XX:+DTraceAllocProbes`. Maybe that's intended or am I missing something?

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

PR: https://git.openjdk.java.net/jdk/pull/6639


More information about the hotspot-compiler-dev mailing list