On Mon, 2 Jun 2025 15:01:15 GMT, Johannes Bechberger <jbechber...@openjdk.org>
wrote:
>> src/hotspot/share/jfr/jfr.inline.hpp line 41:
>>
>>> 39: inline void Jfr::check_and_process_sample_request(JavaThread* jt) {
>>> 40: JfrThreadLocal* tl = jt->jfr_thread_local();
>>> 41: bool has_cpu_time_sample_request = tl->has_cpu_time_jfr_requests();
>>
>> Why this change?
>
> So I don't read the ` tl->has_cpu_time_jfr_requests()` twice on the hot-path
Ok, for now. We should try to come up with a better split.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25302#discussion_r2121466027
