On Mon, 29 Apr 2024 18:48:53 GMT, Adam Sotona <asot...@openjdk.org> wrote:

> Hi,
> During performance optimization work on Class-File API as JDK lambda 
> generator we found some static initialization killers.
> One of them is `java.lang.classfile.Attributes` with tens of static fields 
> initialized with individual attribute mappers, and common set of all mappers, 
> and static map from attribute names to the mappers.
> 
> I propose to turn all the static fields into lazy-initialized static methods 
> and remove `PREDEFINED_ATTRIBUTES` and `standardAttribute(Utf8Entry name)` 
> static mapping method from the `Attributes` API class.
>  
> Please let me know your comments or objections and please review the 
> [PR](https://github.com/openjdk/jdk/pull/19006) and 
> [CSR](https://bugs.openjdk.org/browse/JDK-8331414), so we can make it into 23.
> 
> Thank you,
> Adam

This pull request has now been integrated.

Changeset: cfdc64fc
Author:    Adam Sotona <asot...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/cfdc64fcb43e3b261dddc6cc6947235a9e76154e
Stats:     2285 lines in 149 files changed: 961 ins; 615 del; 709 mod

8331291: java.lang.classfile.Attributes class performs a lot of static 
initializations

Reviewed-by: liach, redestad, vromero

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

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

Reply via email to