RFR: 8353341: C2: removal of a Mod[DF]Node crashes when the node is already dead [v2]

[Marc Chevalier]

> If the Mod[DF]Node has no control projection when it's being removed (because its result is unused), `extract_projections` will fail an assert. So, let's skip the removal.
> 
> But that should happen only when the nodes are already unreachable (control input being transitively top). At the end of the day, the node should be dropped. because of that, so there is no rush, and let dead node deletion do the job.
> 
> On the reduced reproducer, the crash is not common (even with `-XX:RepeatCompilation=300`, it might need more than a run to reproduce). So I've tried my fix on multiple thousands repeat compilations (by 300 packs) without a crash, and without having the modulo node alive at the end.
> 
> Thanks,
> Marc

Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:

  Address review comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24375/files
  - new: https://git.openjdk.org/jdk/pull/24375/files/2a347bc0..f1f0b93b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24375&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24375&range=00-01

  Stats: 6 lines in 2 files changed: 0 ins; 2 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/24375.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24375/head:pull/24375

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