RFR: 8343436: Regression in StackMapGenerator after JDK-8339205
Adam Sotona
asotona at openjdk.org
Fri Nov 1 15:38:26 UTC 2024
On Fri, 1 Nov 2024 15:03:51 GMT, Adam Sotona <asotona at openjdk.org> wrote:
> JDK-8339205 introduced multiple regressions in StackMapGenerator and added a lot of new untested code.
>
> This patch reverts StackMapGenerator to the state before JDK-8339205 and adds one regression test.
>
> Please review.
>
> Thanks,
> Adam
This one test is added to cover the reported bug.
However better test coverage should be always included when statically expanding existing functionality and introducing a lot of new code. It is much smaller chance to catch the bug with existing tests, when each new method is handling only very small set of cases.
Also overriding fields with local variables with modified values is very bad practice.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21835#issuecomment-2452083528
More information about the core-libs-dev
mailing list