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

2023-05-05 Thread Adam Sotona
On Thu, 4 May 2023 16:19:04 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

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

2023-05-05 Thread Adam Sotona
On Fri, 5 May 2023 02:50:45 GMT, Chen Liang wrote: > Since we want an option to toggle stackmap generation, will you add it to the > Classfile options as a temporary measure, before we keep track of these > options in a stateful object (with hierarchy resolver etc.) like brian > envisioned?

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

2023-05-04 Thread Chen Liang
On Thu, 4 May 2023 16:19:04 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

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

2023-05-04 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