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

Coleen Phillimore coleenp at openjdk.org
Thu Jul 7 15:16:40 UTC 2022


On Wed, 6 Jul 2022 19:09:09 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Do not remove Forte::register_stub as it is used on Linux as well
>
> 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?

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

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


More information about the serviceability-dev mailing list