Integrated: 8350115: [leyden] Clean up name_and_sig_as_C_string uses
Aleksey Shipilev
shade at openjdk.org
Fri Feb 14 18:04:27 UTC 2025
On Fri, 14 Feb 2025 16:26:05 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Now that we have optimized a lot of unnecessary compilations, profiles show that various `name_and_sig_as_C_string` are taking about 1.2% in Leyden benchmark profiles. We should sweep those up into conditional log statements and/or common the paths where we actually need these names outside of logging.
>
> After this fix, the profile impact of `name_and_sig_as_C_string` is `<0.05%`.
>
> Performance impact on benchmarks is currently hidden by benchmark noise.
This pull request has now been integrated.
Changeset: e13652b8
Author: Aleksey Shipilev <shade at openjdk.org>
Committer: Vladimir Kozlov <kvn at openjdk.org>
URL: https://git.openjdk.org/leyden/commit/e13652b8371d14f2afd551e357701f24ce39ee61
Stats: 16 lines in 1 file changed: 5 ins; 3 del; 8 mod
8350115: [leyden] Clean up name_and_sig_as_C_string uses
Reviewed-by: asmehra, kvn
-------------
PR: https://git.openjdk.org/leyden/pull/43
More information about the leyden-dev
mailing list