RFR: 8216424: Remove or clean up TimeLivenessAnalysis

[Vladimir Kozlov]

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