RFR: 8158639: C2 compilation fails with SIGSEGV

[Vladimir Kozlov]

Jamsheed, code changes are good. But why you excluded run() method from compilation? The problem happened exactly dutin 
this method compilation.

Thanks,
Vlaidimir

On 8/22/16 9:09 PM, Jamsheed C m wrote:
> Hi Vladimir,
>
> revised webrev with comments: http://cr.openjdk.java.net/~jcm/8158639/webrev.01/index.html
>
> i had updated pit link yesterday night itself, but somehow it didn't work. updated it now again.
>
> Best Regards,
>
> Jamsheed
>
>
> On 8/22/2016 11:36 PM, Vladimir Kozlov wrote:
>> Changes seems fine. Please, add comments in changes.
>>
>> Jamsheed, did you run RBT? I don't see a link in the bug report.
>>
>> Thanks,
>> Vladimir
>>
>> On 8/22/16 5:28 AM, Jamsheed C m wrote:
>>> Hi,
>>>
>>> bug id:https://bugs.openjdk.java.net/browse/JDK-8158639 <https://bugs.openjdk.java.net/browse/JDK-8158639>
>>> webrev:http://cr.openjdk.java.net/~jcm/8158639/webrev.00/ <http://cr.openjdk.java.net/%7Ejcm/8158639/webrev.00/>
>>>
>>> Request for review.
>>>
>>> Description
>>> _linkTo* adapter elimination  in C2 cause wrong jvms  for null checks at callsite. which makes MemberName missing for
>>> trap and reexecute at these sites.
>>>
>>> and in reported bug, this makes wrong data getting loaded in profile data for trap and reexecute.and later getting
>>> crash at recompilation.
>>>
>>> Fix: fix the jvms based on _linkTo*(Declared method signature ) at these trap sites.
>>>
>>>
>>> Best Regards,
>>>
>>> Jamsheed
>>>
>>>
>>>
>