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