RFR: 8292584: assert(cb != __null) failed: must be with -XX:-Inline [v6]

[Vladimir Kozlov]

On Thu, 1 Sep 2022 03:02:50 GMT, Dean Long <dlong at openjdk.org> wrote:

>> generate_Continuation_doYield_entry() creates an interpreter entry point, but jumps to a compiled stub, which needs to be walkable.  The interpreter entry does not create an interpreter frame, so frame walking expects a compiled frame.  Normally everything is OK, but if C1 does not inline the intrinsic and we get to the interpreter entry through the c2i adapter, then things can break if the c2i adapter padded the stack because of alignment.  The easiest fix is to undo what the c2i adapter might have done.
>
> Dean Long has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains eight commits:
> 
>  - merge fix
>  - Merge master
>  - fix build failures
>  - fix zero build
>  - cleanup
>  - fix failed assert
>  - version 2, make doYield a native intrinsic like enterSpecial
>  - fix generate_Continuation_doYield_entry

Good.

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

Marked as reviewed by kvn (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9974