Request for review(S): 7003554: (tiered) assert(is_null_object() || handle() != NULL) failed: cannot embed null pointer

[Vladimir Kozlov]

good.

Vladimir

On 12/1/10 9:07 PM, Igor Veresov wrote:
> On 12/1/10 8:13 PM, Vladimir Kozlov wrote:
>> ciMethod::build_method_data() will return false if it already has
>> method_data.
>
> Right, my bad, fixed:
>
> http://cr.openjdk.java.net/~iveresov/7003554/webrev.01/
>
> igor
>>
>> Vladimir
>>
>> Igor Veresov wrote:
>>> C1 with profiling doesn't check whether the MDO has been really
>>> allocated, which can silently fail if the perm gen is full. The
>>> solution is to check if the allocation failed and bailout out of
>>> inlining or compilation.
>>>
>>>
>>> Webrev: http://cr.openjdk.java.net/~iveresov/7003554/webrev.00/
>>>
>>>
>>> Tested with the failing nightly test, specjvm98, and JPRT.
>>>
>>> Thanks,
>>> igor
>