[code-reflection] RFR: Bytecode round 9 [v5]

Adam Sotona asotona at openjdk.org
Mon Jul 29 15:35:02 UTC 2024


> Bytecode round 9 contains following improvements:
> - `String` (and non-`String`) concats are lifted to `CoreOp.ConcatOp`
> - `LocalsTypeMapper` is fixed to use instruction index as a key (instead of the instruction identity)
> - `LocalsTypeMapper` newly calculates types of uninitialized stack map frame entries and provides that info to `BytecodeLift`
> - `BytecodeLift` is fixed to handle deep stacks correctly while branching + tests added
> - Oboslete `BytecodeGenerator::conversion` method is replaced by direct call of `CodeBuilder::conversion`
> - LocalTypeMapper performs iterative correction of int stack map frame entries to support byte, boolean, short and char types
> 
> `TestSmallCorpus` is not experiencing any first-lift exceptions after the above changes, however we cannot yet say that all the lifted models are correct.
> 
> Please review.
> 
> Thanks,
> Adam

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

 - TestSmallCorpus progress
 - corrected stack to block parametrs conversion
 - fixed SlotSSA to avoid explicit cast of primitive types

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

Changes:
  - all: https://git.openjdk.org/babylon/pull/192/files
  - new: https://git.openjdk.org/babylon/pull/192/files/e2c90bec..81de2334

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=babylon&pr=192&range=04
 - incr: https://webrevs.openjdk.org/?repo=babylon&pr=192&range=03-04

  Stats: 36 lines in 3 files changed: 11 ins; 12 del; 13 mod
  Patch: https://git.openjdk.org/babylon/pull/192.diff
  Fetch: git fetch https://git.openjdk.org/babylon.git pull/192/head:pull/192

PR: https://git.openjdk.org/babylon/pull/192


More information about the babylon-dev mailing list