Fix #2779

[nomisRev]

This PR fixes the actual cause of #2764 & #2760, and replaces the usage of startCoroutineUninterceptedOrReturn with createCoroutineUnintercepted and then manually starting it with resume(Unit).

The usage of startCoroutineUninterceptedOrReturn was correct AFAIK, so the issue would still be with the compiler but it's hard to verify since both startCoroutineUninterceptedOrReturn and createCoroutineUnintercepted are compiler generated methods. So we can not easily inspect the code, or the resulting code since the bytecode. (The bytecode viewer crashes when trying to inspect the code in IDEA).

More details on the YouTrack ticket.

[nomisRev]

This PR reverts the changes from #2765.

[nomisRev]

Fixes #2779

[kyay10]

I'd recommend looking at the bytecode using CFR. I might have a look later to see if the old impl has anything obviously wrong in the bytecode, but I'm really not that familiar with the coroutine implementation details.

[i-walker]

Thanks for the context Looks good
👍🏾

[serras]

I cannot really comment on the implementation (I don't know that much about the coroutine intrinsics), but if the tests seem to cover the behavior that had to be fixed, which is great.