RFR: 8341510: Optimize StackMapGenerator::processFieldInstructions

Shaojin Wen swen at openjdk.org
Fri Oct 4 07:07:34 UTC 2024


On Fri, 4 Oct 2024 04:59:04 GMT, Chen Liang <liach at openjdk.org> wrote:

>> A small optimization to reduce CodeSize, codeSize reduced from 164 to 140.
>> 
>> 1. Use local currentFrame to avoid multiple getfields
>> 2. Use decStack instead of popStack to reduce array access in popStack
>> 3. Call Util.fieldTypeSymbol to pass in type instead of nameAndType
>
> src/java.base/share/classes/jdk/internal/classfile/impl/Util.java line 234:
> 
>> 232:     }
>> 233: 
>> 234:     public static MethodTypeDesc methodTypeSymbol(NameAndTypeEntry nat) {
> 
> Can you remove this method too, since we have a Utf8Entry version?

`methodTypeSymbol(NameAndTypeEntry nat) ` This method will be removed in PR #21344

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21345#discussion_r1787255216


More information about the core-libs-dev mailing list