RFR (S) 8150514: C1 crashes in Canonicalizer::do_ArrayLength() after fix for JDK-8150102

[Tobias Hartmann]

Hi Aleksey,

On 24.02.2016 10:30, Aleksey Shipilev wrote:
> On 02/24/2016 12:22 PM, Tobias Hartmann wrote:
>> On 24.02.2016 10:10, Aleksey Shipilev wrote:
>>> Hi,
>>>
>>> The C1 Canonicalizer fix for arraylength folding (JDK-8150102) crashes
>>> the VM when array constant is "null", and ArrayConstant is not
>>> available. This happens every so often in nightly testing.
>>>
>>> Here is a quick fix for that:
>>>   http://cr.openjdk.java.net/~shade/8150514/webrev.01/
>>
>> Looks good to me.
> 
> Thanks Tobias!
> 
>>> Testing: regression test in Linux x86_64/fastdebug
>>
>> Please don't use bug numbers for test names/folders but use a name describing the purpose of the test.
> 
> Okay, let's do this instead:
>   http://cr.openjdk.java.net/~shade/8150514/webrev.02/

Looks good!

Best regards,
Tobias

> 
> Cheers,
> -Aleksey
> 
>