Withdrawn: 8275638: GraphKit::combine_exception_states fails with "matching stack sizes" assert
Roland Westrelin
roland at openjdk.java.net
Wed Dec 15 10:25:03 UTC 2021
On Fri, 26 Nov 2021 10:09:29 GMT, Roland Westrelin <roland at openjdk.org> wrote:
> Root cause is identical to 8273165 AFIU: late inline of a virtual call
> can throw from 2 different paths (null check and the call
> itself). That breaks because the logic for exceptions expects the
> stack for all paths that throw exceptions to have the same stack size.
>
> AFAIU, the stack doesn't matter exception handling: either the
> exception is caught by a exception handler and then the stack is
> popped and the exception is pushed or, the exception is rethrown to
> the caller in which case the current stack is also popped (that is the
> jvm state for the current method). As a consequence the fix I propose
> is to ignore the stack in GraphKit::combine_exception_states().
>
> AFAIU, the same fix would work for 8273165 but I left the current work
> around as is: not sure if we want to be conservative for now or not
This pull request has been closed without being integrated.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6572
More information about the hotspot-compiler-dev
mailing list