[code-reflection] RFR: BytecodeGenerator cleanup and switch support [v14]

Adam Sotona asotona at openjdk.org
Thu Dec 4 13:27:56 UTC 2025


> The current `BytecodeGenerator` implementation assumes that code models have already been lowered before generation. Both `SwitchExpressionOp` and `SwitchStatementOp` with all-constant labels are examples of operations that can be lowered an alternative way, allowing `BytecodeGenerator` to produce more efficient bytecode using `tableswitch` and `lookupswitch` instructions. 
> 
> The proposed change shifts the responsibility for lowering these operations into the `BytecodeGenerator` itself, enabling selective lowering during bytecode generation. Consequently, `OpBuilder` and `BytecodeGenerator` tests have been updated to avoid invoking the lower transformation step before the `BytecodeGenerator` is called. 
> 
> A new internal operation, `ConstantLabelSwitchOp`, has been introduced. This acts as an intermediate representation for lowered `SwitchExpressionOp` and `SwitchStatementOp` instances with constant labels. 
> 
> Lowering of `SwitchExpressionOp` and `SwitchStatementOp` into `ConstantLabelSwitchOp` is also a part of this PR.
> 
> Lowering of `BreakOp`, `SwitchFallthroughOp` and nested switches will be implemented in a follow-up PR.
> 
> This PR also makes minor refinements to the BytecodeGenerator post-processing transformations.

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

 - Merge remote-tracking branch 'asotona/bytecodegenerator-cleanup' into bytecodegenerator-cleanup
 - Fix JavaSwitchOp.lower to correctly lower regardless of where default case appears

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

Changes:
  - all: https://git.openjdk.org/babylon/pull/706/files
  - new: https://git.openjdk.org/babylon/pull/706/files/811f061a..82c75258

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=babylon&pr=706&range=13
 - incr: https://webrevs.openjdk.org/?repo=babylon&pr=706&range=12-13

  Stats: 105 lines in 1 file changed: 65 ins; 32 del; 8 mod
  Patch: https://git.openjdk.org/babylon/pull/706.diff
  Fetch: git fetch https://git.openjdk.org/babylon.git pull/706/head:pull/706

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


More information about the babylon-dev mailing list