RFR: 8216424: Remove or clean up TimeLivenessAnalysis

[Tobias Hartmann]

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