RFR: 8339320: Optimize ClassFile Utf8EntryImpl#inflate [v3]

Chen Liang liach at openjdk.org
Wed Sep 25 01:17:36 UTC 2024


On Wed, 25 Sep 2024 01:10:04 GMT, Shaojin Wen <swen at openjdk.org> wrote:

>> A very small optimization, split the large method inflate, split the infrequently used paths into a method inflateCHAR
>
> Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
> 
>  - inflateNonAscii
>  - Merge remote-tracking branch 'upstream/master' into optim_class_file_pool_inflat_202408
>    
>    # Conflicts:
>    #	src/java.base/share/classes/jdk/internal/classfile/impl/AbstractPoolEntry.java
>  - Merge remote-tracking branch 'origin/optim_class_file_pool_inflat_202408' into optim_class_file_pool_inflat_202408
>  - Update src/java.base/share/classes/jdk/internal/classfile/impl/AbstractPoolEntry.java
>    
>    Co-authored-by: ExE Boss <3889017+ExE-Boss at users.noreply.github.com>
>  - Update src/java.base/share/classes/jdk/internal/classfile/impl/AbstractPoolEntry.java
>    
>    Co-authored-by: ExE Boss <3889017+ExE-Boss at users.noreply.github.com>
>  - optimize Utf8EntryImpl inflate
>  - Merge remote-tracking branch 'upstream/master' into optim_class_file_pool_inflat_202408
>    
>    # Conflicts:
>    #	src/java.base/share/classes/jdk/internal/classfile/impl/AbstractPoolEntry.java
>  - Suggestions from @liach
>  - fix build error
>  - optimize Utf8EntryImpl inflate

Thanks for this rename and cleanup. Is `inflateNonAscii` code size less than 325?

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

PR Comment: https://git.openjdk.org/jdk/pull/20767#issuecomment-2372672463


More information about the core-libs-dev mailing list