RFR: 8246347: [JVMCI] Set is_method_handle_invoke flag accordingly when describing scope in jvmciCodeInstaller

[Dean Long]

Looks good!

dl

On 6/7/20 1:06 PM, Yudi Zheng wrote:
> Thanks Dean!
> Here is a revision including your suggestion: 
> http://cr.openjdk.java.net/~yzheng/8246347/webrev.01/
>
> -Yudi
>
>> On 6 Jun 2020, at 11:33, Dean Long <dean.long at oracle.com 
>> <mailto:dean.long at oracle.com>> wrote:
>>
>> I found a problem.  You need to make 
>> CompiledMethod::is_deopt_mh_entry() look like is_deopt_entry() by 
>> adding the JVMCI logic that looks backwards by the size of the call 
>> instruction.
>>
>> dl
>>
>> On 6/4/20 12:03 AM, Yudi Zheng wrote:
>>> I did not push this yet. It might require changes on the Graal side. 
>>> I am still thinking about how to merge.
>>>
>>> -Yudi
>>>
>>>> On 4 Jun 2020, at 01:22, Dean Long <dean.long at oracle.com 
>>>> <mailto:dean.long at oracle.com>> wrote:
>>>>
>>>> Does this require recent Graal change in order to work correctly?
>>>>
>>>> dl
>>>>
>>>> On 6/3/20 3:47 PM, Dean Long wrote:
>>>>> Hi Yudi.  I'm seeing an assert in 
>>>>> test/jdk/java/lang/invoke/CallSiteTest.java with a debug build. 
>>>>> Let me remove my changes and see if it still fails.  What testing 
>>>>> did you do?
>>>>>
>>>>> dl
>>>>>
>>>>> On 6/2/20 9:38 AM, Yudi Zheng wrote:
>>>>>> Hello,
>>>>>>
>>>>>> Please review this patch that sets is_method_handle_invoke flag 
>>>>>> accordingly when describing scope at call site in jvmciCodeInstaller.
>>>>>>
>>>>>> http://cr.openjdk.java.net/~yzheng/8246347/webrev.00/
>>>>>> https://bugs.openjdk.java.net/browse/JDK-8246347
>>>>>>
>>>>>> Many thanks,
>>>>>> Yudi
>>
>