[lworld] Integrated: 8262231: [lworld] C2 compilation fails with assert "user must call transfer_exceptions_into_jvms"

[Tobias Hartmann]

On Tue, 23 Feb 2021 17:11:05 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> When running with `-XX:PerMethodSpecTrapLimit=0 -XX:PerMethodTrapLimit=0`, we hit an assert because the unsafe access intrinsic adds an exception to the `GraphKit` state but then bails out and therefore does not transfer that exception into the `JVMState`. In general, we should only modify the graph once we are sure we won't bail out from intrinsification. I've added an assert that catches this even without any VM flags and fixed all affected intrinsics.
> 
> Best regards,
> Tobias

This pull request has now been integrated.

Changeset: 1816db37
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/1816db37
Stats:     54 lines in 1 file changed: 18 ins; 26 del; 10 mod

8262231: [lworld] C2 compilation fails with assert "user must call transfer_exceptions_into_jvms"

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

PR: https://git.openjdk.java.net/valhalla/pull/348