Request for reviews (S): 6823453: DeoptimizeALot causes fastdebug server jvm to fail with assert(false,"unscheduable graph")

[Vladimir Kozlov]

I added test case suggested by John.
I also replaced the code in GraphKit::new_array
with the _gvn.transform(ccast) call and the result's
type check as John suggested. I think it is safe
to not delay the transform() call since the TOP result
will not be used.

Thanks,
Vladimir


http://cr.openjdk.java.net/~kvn/6823453/webrev.01

Fixed 6823453: DeoptimizeALot causes fastdebug server jvm to fail with assert(false,"unscheduable graph")

Problem:
The code in GraphKit::new_array replaces a negative value with
TOP (CastII(-1)#0) for all uses if it is used as new array length.
Incorrect ideal graph is generated as result.

Solution:
Don't replace a value with TOP.
Added regression test.

Reviewed by:

Fix verified (y/n): y, bug tests

Other testing:
JPRT