Integrated: 8331402: G1: Remove is_active() calls in G1HRPrinter logging

Thomas Schatzl tschatzl at openjdk.org
Fri May 3 12:35:59 UTC 2024


On Thu, 2 May 2024 13:03:09 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   the `is_active()` call for the various G1HRPrinter methods is unnecessary as the `log_trace()` call will do the exact same check again anyway.
> 
> Testing: gha, local compilation
> 
> Based on #19058.
> 
> Hth,
>   Thomas

This pull request has now been integrated.

Changeset: 58ef9e48
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/58ef9e4805c0cc78935eb5a1c82ae10411d52e85
Stats:     62 lines in 1 file changed: 0 ins; 50 del; 12 mod

8331402: G1: Remove is_active() calls in G1HRPrinter logging

Reviewed-by: iwalulya, ayang, gli

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

PR: https://git.openjdk.org/jdk/pull/19061


More information about the hotspot-gc-dev mailing list