RFR (trivial): 8219919: RuntimeStub's name lost with PrintFrameConverterAssembly

[Jie Fu]

Thanks Vladimir.


On 2019年03月01日 08:56, Jie Fu wrote:
> Thank you so much, Vladimir.
>
> I really appreciate if someone could help to sponsor it.
> Thanks in advance.
>
> Best regards,
> Jie
>
> On 2019/3/1 上午3:36, Vladimir Kozlov wrote:
>> Looks good.
>>
>> thanks,
>> Vladimir
>>
>> On 2/28/19 4:29 AM, Jie Fu wrote:
>>> Hi all,
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8219919
>>>
>>> The RuntimeStub's name is lost when dumping C2's runtime stub with 
>>> -XX:+PrintFrameConverterAssembly.
>>> However, it do exist when dumping with -XX:+PrintStubCode.
>>>
>>> It would be more friendly if the stub's name was dumped as well for 
>>> JVM debuggers with -XX:+PrintFrameConverterAssembly.
>>>
>>> It can be fixed by
>>> ---------------------------------------
>>> diff -r 56089cf6152c src/hotspot/share/opto/output.cpp
>>> --- a/src/hotspot/share/opto/output.cpp Tue Feb 26 05:46:02 2019 -0800
>>> +++ b/src/hotspot/share/opto/output.cpp Thu Feb 28 19:52:40 2019 +0800
>>> @@ -1556,6 +1556,8 @@
>>>         }
>>>         if (method() != NULL) {
>>>           method()->print_metadata();
>>> +      } else if (stub_name() != NULL) {
>>> +        tty->print_cr("Generating RuntimeStub - %s", stub_name());
>>>         }
>>>         dump_asm(node_offsets, node_offset_limit);
>>>         if (xtty != NULL) {
>>> ---------------------------------------
>>>
>>> The change has been tested on Linux/x64.
>>> Could you please review it?
>>> Thanks a lot.
>>>
>>> Best regards,
>>> Jie
>>>
>>>
>