RFR: 8216424: Remove or clean up TimeLivenessAnalysis

[Claes Redestad]

Thanks, Vladimir,

I'll go ahead and remove it, then.

/Claes

On 2019-01-13 03:20, Vladimir Kozlov wrote:
> Agree with removal. I never used it.
> 
> Thanks,
> Vladimir
> 
> On 1/9/19 8:54 AM, Tobias Hartmann wrote:
>> Hi Claes,
>>
>> Both webrevs look good to me but I would prefer removal as well. I 
>> haven't ever seen anyone using
>> that flag but let's wait for more opinions.
>>
>> Best regards,
>> Tobias
>>
>> On 09.01.19 16:10, Claes Redestad wrote:
>>> Hi,
>>>
>>> implementation for the develop flag TimeLivenessAnalysis leaves a few
>>> breadcrumbs in product builds (in particular TraceTime
>>> constructors/destructors aren't being inlined, so the compiler doesn't
>>> realize these objects aren't actually doing anything)
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8216424
>>>
>>> This should be either cleaned up:
>>> http://cr.openjdk.java.net/~redestad/8216424/cleanup.00/
>>>
>>> .. or the flag should be removed altogether:
>>> http://cr.openjdk.java.net/~redestad/8216424/remove.00/
>>>
>>> I favor removal since the statistics collected by this analysis does
>>> not seem very useful and any real performance effect could/should be
>>> estimated using real profiling tools on product builds, anyhow.
>>>
>>> Thanks!
>>>
>>> /Claes