RFR: 8341272: Factory to create wide iinc instruction with small arguments [v2]
Trevor Bond
duke at openjdk.org
Wed Dec 10 01:37:05 UTC 2025
> Add a new factory method to `IncrementInstruction` that allows the explicit creation of a wide iinc instruction, even with a `slot` and `constant` that could fit into a normal iinc instruction. Previously, only one factory for iinc instructions existed, which inferred the type of instruction needed given the size of `slot` and `constant`. Add additional test cases for the new factory as well. All tier 1 tests and classfile tests have passed with these changes.
Trevor Bond has updated the pull request incrementally with one additional commit since the last revision:
Clean up javadocs and address other review comments
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/28729/files
- new: https://git.openjdk.org/jdk/pull/28729/files/4acc5b44..44612c5f
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=28729&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=28729&range=00-01
Stats: 33 lines in 3 files changed: 8 ins; 19 del; 6 mod
Patch: https://git.openjdk.org/jdk/pull/28729.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28729/head:pull/28729
PR: https://git.openjdk.org/jdk/pull/28729
More information about the core-libs-dev
mailing list