> 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 provided. 
> 2. StackMapTable attribute is not generated for class file versions below 50.
> 
> StackMapsTest is also extended to test this patch.
> 
> Please review.
> 
> Thanks,
> Adam

Adam Sotona has updated the pull request incrementally with two additional 
commits since the last revision:

 - Update 
src/java.base/share/classes/jdk/internal/classfile/impl/AbstractInstruction.java
   
   Co-authored-by: liach <7806504+li...@users.noreply.github.com>
 - Update src/java.base/share/classes/jdk/internal/classfile/Opcode.java
   
   Co-authored-by: ExE Boss <3889017+exe-b...@users.noreply.github.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/13478/files
  - new: https://git.openjdk.org/jdk/pull/13478/files/473fc8e8..cc2c1059

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=13478&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=13478&range=02-03

  Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/13478.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13478/head:pull/13478

PR: https://git.openjdk.org/jdk/pull/13478

Reply via email to