RFR: 8341512: Optimize StackMapGenerator::processInvokeInstructions [v3]

Chen Liang liach at openjdk.org
Sat Oct 5 15:32:40 UTC 2024


On Sat, 5 Oct 2024 01:27:18 GMT, Shaojin Wen <swen at openjdk.org> wrote:

>> A small optimization for StackMapGenerator::processInvokeInstructions.
>> 
>> 1. Use local currentFrame to avoid multiple getfields
>> 2. remove Util.methodTypeSymbol(NameAndTypeEntry)
>> 3. Use decStack instead of popStack to reduce array access in popStack
>> 4. codeSize reduced from 277 to 262
>
> Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
> 
>  - Merge branch 'master' into optim_classfile_stack_map_process_invoke_202410
>  - Update src/java.base/share/classes/java/lang/classfile/attribute/EnclosingMethodAttribute.java
>    
>    Co-authored-by: ExE Boss <3889017+ExE-Boss at users.noreply.github.com>
>  - use decStack instead of popStack
>  - remove Util.methodTypeSymbol(NameAndTypeEntry)
>  - optimize processInvokeInstructions

Util method removal fix conflict looks good

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

Marked as reviewed by liach (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21344#pullrequestreview-2349649471


More information about the core-libs-dev mailing list