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

[Jie Fu]

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
>>
>>