RFR: 8325485: IncrementInstructions.of(int, int) is not storing the args [v2]
Adam Sotona
asotona at openjdk.org
Fri Feb 9 10:47:08 UTC 2024
> 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 <turbanoff at 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
More information about the core-libs-dev
mailing list