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