RFR: 8263587: C2: JVMS not cloned when needs_clone_jvms() is true

Mon Mar 15 11:22:11 UTC 2021

On Mon, 15 Mar 2021 10:56:51 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> We need a C2 fix to prevent crashes because JVMS doesn't get cloned as needed. See linked issue for details.

Looks good.

Cleanup idea: `CallNode::clone_jvms()` overrides differ only in the guarding condition. Extract node type dependent checks into a new helper method (`CallNode::needs_clone_jvms()`) and use it from now non-virtual `CallNode::clone_jvms()`.

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

Marked as reviewed by vlivanov (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3009