On Mon, 26 Aug 2024 14:54:59 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> Claes Redestad has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Copyright
>
> src/java.base/share/classes/jdk/internal/classfile/impl/ClassFileImpl.java 
> line 86:
> 
>> 84:         var amo = attributeMapperOption;
>> 85:         for (var o : options) {
>> 86:             if (o instanceof StackMapsOption oo) {
> 
> Just out of curiosity, will this be slower than the switch statement?

It may after `SwitchBootstraps` is improved; but this code will be relied on by 
`SwitchBootstraps` to generate switches, so it won't be able to use pattern 
matching switches.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20714#discussion_r1731382984

Reply via email to