Re: RFR: 8305990: Stripping debug info of ASM 9.5 fails [v2]

2023-04-20 Thread Adam Sotona
On Wed, 19 Apr 2023 13:30:44 GMT, Chen Liang wrote: >> Adam Sotona has updated the pull request incrementally with one additional >> commit since the last revision: >> >> DiscontinuedInstruction implementation + test > > src/java.base/share/classes/jdk/internal/classfile/impl/DirectCodeBuilde

Re: RFR: 8305990: Stripping debug info of ASM 9.5 fails [v2]

2023-04-19 Thread Adam Sotona
On Wed, 19 Apr 2023 13:06:20 GMT, Adam Sotona wrote: >> Classfile API didn't handle transformations of class files version 50 and >> below correctly. >> >> Proposed fix have two parts: >> 1. Inflation of branch targets does not depend on StackMapTable attribute >> presence for class file ver

Re: RFR: 8305990: Stripping debug info of ASM 9.5 fails [v2]

2023-04-19 Thread Chen Liang
On Wed, 19 Apr 2023 13:06:20 GMT, Adam Sotona wrote: >> Classfile API didn't handle transformations of class files version 50 and >> below correctly. >> >> Proposed fix have two parts: >> 1. Inflation of branch targets does not depend on StackMapTable attribute >> presence for class file ver

Re: RFR: 8305990: Stripping debug info of ASM 9.5 fails [v2]

2023-04-19 Thread Adam Sotona
> Classfile API didn't handle transformations of class files version 50 and > below correctly. > > Proposed fix have two parts: > 1. Inflation of branch targets does not depend on StackMapTable attribute > presence for class file version 50 and below. Alternative fallback > implementation is