On Wed, 6 Jul 2022 19:09:09 GMT, Coleen Phillimore <cole...@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

Reply via email to