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

Serguei Spitsyn sspitsyn at openjdk.org
Sat Jul 9 01:04:46 UTC 2022


On Fri, 8 Jul 2022 23:06:03 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> src/hotspot/share/runtime/sharedRuntime.cpp line 2700:
>> 
>>> 2698:     if (JvmtiExport::should_post_dynamic_code_generated()) {
>>> 2699:       JvmtiExport::post_dynamic_code_generated(blob_id, new_adapter->content_begin(), new_adapter->content_end());
>>> 2700:     }
>> 
>> The lines 2698-2670 is better to move out of the if-statement at the line 2687.
>
> HI Serguei, thanks for the review. Lines 2698-2700 need the blob_id which is generated at line 2688, so they have to stay inside the outer "if" block.

Thanks, Ioi. You are right.

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

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


More information about the serviceability-dev mailing list