<i18n dev> RFR: 8335252: Reduce size of j.u.Formatter.Conversion#isValid [v4]
duke
duke at openjdk.org
Fri Jun 28 16:22:20 UTC 2024
On Fri, 28 Jun 2024 12:46:49 GMT, Shaojin Wen <duke at openjdk.org> wrote:
>> Currently, the java.util.Formatter$Conversion::isValid method is implemented based on switch, which cannot be inlined because codeSize > 325. This problem can be avoided by implementing it with ImmutableBitSetPredicate.
>>
>> use `-XX:+UnlockDiagnosticVMOptions -XX:+PrintInlining` to see the master branch:
>>
>> @ 109 java.util.Formatter$Conversion::isValid (358 bytes) failed to inline: hot method too big
>>
>>
>> current version
>>
>> @ 109 java.util.Formatter$Conversion::isValid (10 bytes) inline (hot)
>> @ 4 jdk.internal.util.ImmutableBitSetPredicate$SmallImmutableBitSetPredicate::test (50 bytes) inline (hot)
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>
> fix comment
@wenshao
Your change (at version 5707dc675b5b7d4960d3e3121a5c8fe341862d7b) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/19926#issuecomment-2197261960
More information about the i18n-dev
mailing list