RFR(S): 8005079: fix LogCompilation for incremental inlining

[Roland Westrelin]

Hi Vladimir,

>>> It is good.
>> 
>> Thanks for the review.
>> 
>>> One thing you may need to change is C->log_inline_id((jlong)this).
>>> Address is not good id when you look on log file and it could be the same between compilations.
>>> I think some combination of compile_id<<shift+(adjustment for OSR when CICountOSR=true)+(unique node number at time when cg was created or other sequential number).
>> 
>> I don’t understand why it matters. The id is only used within a single compilation by the LogCompilation tool to correctly match a call site at which we stopped inlining with a call site at which we start to inline again later on.
> 
> It will be confusing when you search in a log file for such id and it matches ids from other compilations.

I followed your suggestion:
http://cr.openjdk.java.net/~roland/8005079/webrev.04/

Roland.

> 
> Vladimir
> 
>> 
>> Roland.
>> 
>>> Thanks,
>>> vladimir
>>> 
>>> On 3/21/14 2:04 AM, Roland Westrelin wrote:
>>>>>> http://cr.openjdk.java.net/~roland/8005079/webrev.02/
>>>>> 
>>>>> In order for equals to be correct you have to do an instanceof check.
>>>> 
>>>> Ok.
>>>> 
>>>> http://cr.openjdk.java.net/~roland/8005079/webrev.03/
>>>> 
>>>> Roland.
>>>> 
>>