[aarch64-port-dev ] git: openjdk/aarch64-port: master: 6232281: -XX:-UseLoopSafepoints causes assert(v_false, "Parse::remove_useless_nodes missed this node")

[Tobias Hartmann]

Changeset: eb35ade9
Author:    Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
Date:      2020-11-20 12:06:36 +0000
URL:       https://github.com/openjdk/aarch64-port/commit/eb35ade9

6232281: -XX:-UseLoopSafepoints causes assert(v_false,"Parse::remove_useless_nodes missed this node")

Check for nodes missed by remove_useless_nodes() only if PhaseRemoveUseless has
actually been run. This makes it possible to use -XX:-UseLoopSafepoints without
crashing trivially, although implicit assumptions in other parts of C2 about the
existence of loop safepoints might lead to more subtle failures for more complex
methods.

Reviewed-by: neliasso, thartmann, kvn

! src/hotspot/share/opto/phaseX.cpp
+ test/hotspot/jtreg/compiler/arguments/TestDisableUseLoopSafepoints.java