RFR: 8342818: Implement CPU Time Profiling for JFR [v4]

David Holmes dholmes at openjdk.org
Fri Nov 1 05:41:38 UTC 2024


On Wed, 30 Oct 2024 18:10:52 GMT, Johannes Bechberger <jbechberger at openjdk.org> wrote:

>> src/hotspot/share/jfr/recorder/stacktrace/jfrAsyncStackTrace.cpp line 56:
>> 
>>> 54:   }
>>> 55:   assert(current_thread->in_asgct(), "invariant");
>>> 56:   assert(jt != current_thread || current_thread->in_asgct(), "invariant");
>> 
>> The second clause of the second assert is redundant given the first assert.
>
> The second assert is redundant.

Good point :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20752#discussion_r1825445850


More information about the hotspot-dev mailing list