RFR: 8289780: Avoid formatting stub names when Forte is not enabled [v2]

Ioi Lam iklam at openjdk.org
Fri Jul 8 18:37:42 UTC 2022


On Thu, 7 Jul 2022 15:12:49 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/prims/forte.hpp line 32:
>> 
>>> 30: class Forte : AllStatic {
>>> 31:  public:
>>> 32:    static bool is_enabled() NOT_JVMTI_RETURN_(false);
>> 
>> I don't think the rest of this forte code is disabled by JVMTI.
>
> If the answer to whether it's enabled is something you want to be fast, and doesn't change, maybe make it check a variable?

The code in the `Forte` class, as well as the non-trivial of `AsyncGetCallTrace()` in forte.cpp, are inside `#if INCLUDE_JVMTI`. That's why I use `NOT_JVMTI_RETURN_` for the new function.

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

PR: https://git.openjdk.org/jdk/pull/9386


More information about the serviceability-dev mailing list