RFR: JDK-8213480: update internal ASM version to 7.0

Vicente Romero vicente.romero at oracle.com
Thu Nov 8 16:39:39 UTC 2018


Hi David, Igor

On 11/7/18 10:03 PM, David Holmes wrote:
> Hi Vicente,
>
> All of the javadoc comment reformatting makes it nearly impossible to 
> see the actual substantive changes :(
>
> ASM 7 also supports the Nestmate attributes and I was trying to see 
> how/where that appeared but its somewhat obscure. Oh well.
>
> Is it that case that the code the uses the ASM library, like the JFR 
> code and jlink code, and the tests, doesn't actually _have to_ change 
> to specifying Opcodes.ASM7 unless they plan on using ASM7 features?

I changed only the tests for which the new ASM was complaining about a 
particular API available only for ASM7

> If so then you could split out the actual update of ASM from the 
> updates to the users of ASM (some of which may be quite fine with ASM5).

I have made two webrevs to make the review easier [1], contain only the 
changes to the internal asm and [2] contains the changes to the clients 
plus make files, legal, etc. I have also made the changes to 
ClassWriterExt and affected test proposed by Igor in another mail,

>
> Thanks,
> David

Thanks,
Vicente

[1] http://cr.openjdk.java.net/~vromero/8213480/webrev.asm.7.only.00/
[2] 
http://cr.openjdk.java.net/~vromero/8213480/webrev.asm.7.additional.changes.00/
>
> On 8/11/2018 1:56 AM, Vicente Romero wrote:
>> Hi,
>>
>> Version 7.0 of ASM has been released. This version supports condy, 
>> yay!, and we want to include it in JDK 12. Please review [1] which 
>> includes:
>> - the new version perse substituting the preview ASM internal version 
>> in the JDK
>> - changes to additional files in particular some tests, mostly 
>> hotspot tests.
>>
>> Thanks,
>> Vicente
>>
>> [1] http://cr.openjdk.java.net/~vromero/8213480/webrev.00/



More information about the hotspot-dev mailing list