RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

Eric Caspole-2
Hi everyone,
Could you review this small enahncement to the LogCompilation utility so
it would show the internal intrinsic name similarly to +PrintIntrinsics.
Thanks,
Eric

JBS:
https://bugs.openjdk.java.net/browse/JDK-8192857

Webrev:
http://cr.openjdk.java.net/~ecaspole/JDK-8192857/02/02/webrev/
Reply | Threaded
Open this post in threaded view
|

Re: RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

Vladimir Kozlov
Hi Eric,

Method 'm' is not used:

+            Method m = cs.getMethod();
+            cs.setIntrinsicName(id);

Thanks,
Vladimir

On 1/2/18 11:59 AM, Eric Caspole wrote:

> Hi everyone,
> Could you review this small enahncement to the LogCompilation utility so it would show the internal
> intrinsic name similarly to +PrintIntrinsics.
> Thanks,
> Eric
>
> JBS:
> https://bugs.openjdk.java.net/browse/JDK-8192857
>
> Webrev:
> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/02/02/webrev/
Reply | Threaded
Open this post in threaded view
|

Re: RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

Eric Caspole-2
Whoops, good catch.
New one:

http://cr.openjdk.java.net/~ecaspole/JDK-8192857/03/webrev/

Eric

On 01/02/2018 06:49 PM, Vladimir Kozlov wrote:

> Hi Eric,
>
> Method 'm' is not used:
>
> +            Method m = cs.getMethod();
> +            cs.setIntrinsicName(id);
>
> Thanks,
> Vladimir
>
> On 1/2/18 11:59 AM, Eric Caspole wrote:
>> Hi everyone,
>> Could you review this small enahncement to the LogCompilation utility
>> so it would show the internal intrinsic name similarly to
>> +PrintIntrinsics.
>> Thanks,
>> Eric
>>
>> JBS:
>> https://bugs.openjdk.java.net/browse/JDK-8192857
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/02/02/webrev/
Reply | Threaded
Open this post in threaded view
|

Re: RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

Vladimir Kozlov
Good.

Thanks,
Vladimir

On 1/3/18 7:20 AM, Eric Caspole wrote:

> Whoops, good catch.
> New one:
>
> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/03/webrev/
>
> Eric
>
> On 01/02/2018 06:49 PM, Vladimir Kozlov wrote:
>> Hi Eric,
>>
>> Method 'm' is not used:
>>
>> +            Method m = cs.getMethod();
>> +            cs.setIntrinsicName(id);
>>
>> Thanks,
>> Vladimir
>>
>> On 1/2/18 11:59 AM, Eric Caspole wrote:
>>> Hi everyone,
>>> Could you review this small enahncement to the LogCompilation utility so it would show the internal intrinsic name
>>> similarly to +PrintIntrinsics.
>>> Thanks,
>>> Eric
>>>
>>> JBS:
>>> https://bugs.openjdk.java.net/browse/JDK-8192857
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/02/02/webrev/
Reply | Threaded
Open this post in threaded view
|

Re: RFR [11] (XS): 8192857: LogCompilation could show the intrinsics more like +PrintIntrinsics

George Triantafillou
+1

-George

On 1/3/2018 1:11 PM, Vladimir Kozlov wrote:

> Good.
>
> Thanks,
> Vladimir
>
> On 1/3/18 7:20 AM, Eric Caspole wrote:
>> Whoops, good catch.
>> New one:
>>
>> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/03/webrev/
>>
>> Eric
>>
>> On 01/02/2018 06:49 PM, Vladimir Kozlov wrote:
>>> Hi Eric,
>>>
>>> Method 'm' is not used:
>>>
>>> +            Method m = cs.getMethod();
>>> +            cs.setIntrinsicName(id);
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 1/2/18 11:59 AM, Eric Caspole wrote:
>>>> Hi everyone,
>>>> Could you review this small enahncement to the LogCompilation
>>>> utility so it would show the internal intrinsic name similarly to
>>>> +PrintIntrinsics.
>>>> Thanks,
>>>> Eric
>>>>
>>>> JBS:
>>>> https://bugs.openjdk.java.net/browse/JDK-8192857
>>>>
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~ecaspole/JDK-8192857/02/02/webrev/