RFR: 8282590: C2: assert(addp->is_AddP() && addp->outcnt() > 0) failed: Don't process dead nodes [v2]

[Emanuel Peter]

> Problem:
> Sometimes nodes are generated, and then not properly added to any other node, nor added to the worklist, so that it could be deleted. In some cases, a node can thus survive after IGVN and enter into `ConnectionGraph::compute_escape`, where no dead nodes are expected (except constants).
> 
> In `ArrayCopyNode::prepare_array_copy`:
> In one, I now add the new ConvI2LNode to the worklist if we are aborting.
> In the other case, I could safely reorder the code, such new ConvI2LNode would only be generated once we know we are not going to abort.
> 
> In `SubTypeCheckNode::Ideal`:
> Replaced `set_req` with `set_req_X`, such that the input-node that is replaced (which is now potentially a dead node) is put on the worklist.
> 
> Running Tests...
> Wanted to add regression test, but that one now fails sadly.

Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:

  added regression test, and fixed a small similar bug as the others

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7728/files
  - new: https://git.openjdk.java.net/jdk/pull/7728/files/513f6ea9..95478505

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7728&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7728&range=00-01

  Stats: 22 lines in 2 files changed: 22 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7728.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7728/head:pull/7728

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