[10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

[Rahul Raghavan]

Thank you Mandy for review.

On Friday 24 November 2017 11:11 PM, mandy chung wrote:
> Looks good.  Thanks for fixing the test, Rahul.
> 
> Mandy
> 
> On 11/24/17 3:16 AM, Rahul Raghavan wrote:
>> Hi,
>>
>> Please review the following fix proposal for 8191813.
>>
>> webrev - http://cr.openjdk.java.net/~rraghavan/8191813/webrev.00/
>>
>>
>> jbs - https://bugs.openjdk.java.net/browse/JDK-8191813
>>
>> -- reported issue - After the latest sync from jdk/jdk the test 
>> SpreadNullArg started to fail in tier1 on all platforms.
>>
>> -- Found this SpreadNullArg.java failure Caused after JDK-8157246 fix.
>> jbs - https://bugs.openjdk.java.net/browse/JDK-8157246.
>> 'MHs.arrayLength, arrayElementGetter/Setter, arrayConstructor need to 
>> specify invocation-time behavior'.
>> changeset - http://hg.openjdk.java.net/jdk/jdk/rev/76519338df34
>>
>> - So now with 8157246 fix, if method handle is invoked with a null 
>> array reference, NullPointerException will be thrown. (Not 
>> IllegalArgumentException)
>>
>> - So above fix proposal in SpreadNullArg.java to expect 
>> NullPointerException after 8157246 changes.
>>
>>
>> Thanks,
>> Rahul
>