[code-reflection] RFR: Unreflect tests [v3]

Paul Sandoz psandoz at openjdk.org
Thu Dec 18 18:11:27 UTC 2025


On Thu, 18 Dec 2025 14:44:14 GMT, Adam Sotona <asotona at openjdk.org> wrote:

>> This PR implement `Unreflect` test tool and re-runs tests in the unreflected form.
>> 
>> `TestSwitchExpressionOp` and `TestSwitchStatementOp` yet to be temporary excluded as failing  (see [JDK-8373573](https://bugs.openjdk.org/browse/JDK-8373573)).
>
> Adam Sotona has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - obsolete debug print
>    
>  - new line

This is a very nice approach. There are some tests where reflectable methods are never invoked e.g., `CoreBinaryOpsTest`, which tests the model in the interpreter and explicitly generated bytecode, in this case we arguably should also test direct invocation So i think it may be worth going through and checking as not all tests need to be modified and some might be to ensure larger converage. Further, with this test do we need as many specific bytecode tests? can we simplify or reduce those now?

Any thoughts on how we might apply the same approach to reflectable lambdas? I guess we can rewrite the invoke dynamic instructions.

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

PR Review: https://git.openjdk.org/babylon/pull/762#pullrequestreview-3594294003


More information about the babylon-dev mailing list