RFR: 8289780: Avoid formatting stub names when Forte is not enabled [v2]
Ioi Lam
iklam at openjdk.org
Fri Jul 8 23:09:46 UTC 2022
On Fri, 8 Jul 2022 22:53:49 GMT, Serguei Spitsyn <sspitsyn 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/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.
-------------
PR: https://git.openjdk.org/jdk/pull/9386
More information about the hotspot-dev
mailing list