RFR: 8373122: JFR build failure with CDS disabled due to -Werror=unused-function after JDK-8365400 [v2]

Markus Grönlund mgronlun at openjdk.org
Tue Dec 23 07:39:56 UTC 2025


On Tue, 23 Dec 2025 03:33:08 GMT, Hao Sun <haosun at openjdk.org> wrote:

>> `get_source(const AOTClassLocation* cl, JavaThread* jt)` is only used by `JfrClassDefineEvent::on_restoration()` within an `INCLUDE_CDS` guard.
>> 
>> Add the same `INCLUDE_CDS` guard to mitigate the GCC warning.
>> 
>> Tests: JDK build with CDS disabled passed on both x86_64 and aarch64.
>>        tier1~3 passed on both x86_64 and aarch64.
>
> Hao Sun has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Move down the helper to avoid introducing new conditions
>   
>   Address mgronlun's comment

Marked as reviewed by mgronlun (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/28917#pullrequestreview-3606957792


More information about the build-dev mailing list