[code-reflection] RFR: Lower SwitchExpressionOp [v5]

Paul Sandoz psandoz at openjdk.org
Wed May 29 19:45:25 UTC 2024


On Wed, 29 May 2024 01:49:46 GMT, Mourad Abbay <mabbay at openjdk.org> wrote:

>> Lowering of SwitchExpressionOp
>
> Mourad Abbay has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
> 
>  - Merge branch 'code-reflection' into lower-switch-expr
>    
>    # Conflicts:
>    #	src/java.base/share/classes/java/lang/reflect/code/op/ExtendedOp.java
>  - Generate lowered form that conforms to the JLS when selector expression is null
>  - Refactor
>  - Remove unnecessary code
>  - Remove unnecessary comments
>  - Make JavaSwitchFallThroughOp lowerable
>  - Remove params of blocks that represents label bodies
>  - Fix a typo in the doc
>  - More tests
>  - rename variable to avoid confusion
>  - ... and 2 more: https://git.openjdk.org/babylon/compare/331553ef...98e52027

This looks good. Just some minor code changes. Subsequent PRs can add more test cases.

test/jdk/java/lang/reflect/code/TestSwitchExpressionOp.java line 118:

> 116:                 return block;
> 117:             }
> 118:         });

Suggestion:

        CoreOp.FuncOp lf = f.transform(OpTransformer.LOWERING_TRANSFORMER);

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

Marked as reviewed by psandoz (Lead).

PR Review: https://git.openjdk.org/babylon/pull/45#pullrequestreview-2086249843
PR Review Comment: https://git.openjdk.org/babylon/pull/45#discussion_r1619387653


More information about the babylon-dev mailing list