On Fri, 15 May 2026 21:08:46 GMT, Vladimir Ivanov <[email protected]> wrote:

>> Manuel Hässig has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 49 commits:
>> 
>>  - Merge branch 'master' into remove-jvmci
>>  - Remove graal from test/langtools
>>  - Remove jvmci and graal from test/lib
>>  - Remove jvmci and graal from test/jdk/jdk/jfr
>>  - Remove jvmci from test/jdk/tools/jlink
>>  - Remove jvmci and graal from test/jdk/jdk/modules
>>  - Remove jvmci and graal frmo test/jdk/java/lang
>>  - Remove graal from test/jdk/javax
>>  - Remove jvmci and graal from test/jdk/com/sun/jdi
>>  - Remove jvmci and graal from test/hotspot/jtreg/runtime
>>  - ... and 39 more: https://git.openjdk.org/jdk/compare/061bd076...0100264f
>
> src/hotspot/share/compiler/compileBroker.cpp line 571:
> 
>> 569: static void register_jfr_phasetype_serializer() {
>> 570:   ResourceMark rm;
>> 571:   static bool first_registration = true;
> 
> `first_registration` is unused.

Good catch! Removed with 35b2f1a7423.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30834#discussion_r3257668798

Reply via email to