RFR: 8316967: Remove unnecessary vmtimer from UnregisteredClasses::load_class

David Holmes dholmes at openjdk.org
Thu Oct 5 03:13:21 UTC 2023


On Wed, 4 Oct 2023 23:30:58 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Simple fix for removing the vmtimer from `UnregisteredClasses::load_class` and the corresponding `PerfClassTraceTime` constructor.
> 
> Passed tier1 testing.

As it stands this timer doesn't measure anything, but presumably the intent was to measure the execution time of this method? So why not just fix the scope of the timer rather than removing it?

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

PR Comment: https://git.openjdk.org/jdk/pull/16046#issuecomment-1747968934


More information about the hotspot-runtime-dev mailing list