Request for reviews (S): 6823453: DeoptimizeALot causes fastdebug server jvm to fail with assert(false,"unscheduable graph")
Vladimir.Kozlov at Sun.COM
Fri Apr 3 11:31:44 PDT 2009
Thank you, Tom
It passed CTW and JPRT, so I am going to push it.
Tom Rodriguez wrote:
> That looks much nicer.
> On Apr 2, 2009, at 7:32 PM, Vladimir Kozlov wrote:
>> How about this
>> I also replaced the call remove_globally_dead_node()
>> with igvn->replace_node(catchproj, phase->C->top()).
>> It can't be used for nodes with out edges (kills regions otherwise).
>> Tom Rodriguez wrote:
>>> I think this:
>>> + Node* catchproj =
>>> has to be coded more carefully. That's the shape of a well-formed
>>> fall through edge but there's no guarantee it's well formed. It's
>>> too bad the extract_call_projections logic isn't packaged up a little
>>> On Apr 2, 2009, at 3:39 PM, Vladimir Kozlov wrote:
>>>> New (fourth?) version of the fix suggested by Tom.
>>>> I added the test case when array length is not defined
>>>> until loop optimizations. Only this version can compile it.
More information about the hotspot-compiler-dev