> ClassFile API provides two sets of instructions implementations (bound and 
> unbound).
> Unbound implementation of `IncrementInstruction::constant` returns invalid 
> value. 
> This bug discovered a hole in the ClassFile API test coverage.
> 
> This patch provides very simple fix of `IncrementInstruction`
> and more complex fix of the test framework to cover all unbound instruction 
> with automated tests.
> 
> The test framework fix includes correction of hash calculation of 
> instructions in `ClassRecord` and
> two-pass transformation in `RebuildingTransformation`. Second pass has been 
> added to discover bugs in unbound-to-unbound instruction conversions.
> 
> Please review.
> 
> Thanks,
> Adam

Adam Sotona has updated the pull request incrementally with one additional 
commit since the last revision:

  Update test/jdk/jdk/classfile/helpers/RebuildingTransformation.java
  
  Co-authored-by: Andrey Turbanov <turban...@gmail.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17770/files
  - new: https://git.openjdk.org/jdk/pull/17770/files/92d5af38..8586d628

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17770&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17770&range=00-01

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

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

Reply via email to