RFR: 8216424: Remove or clean up TimeLivenessAnalysis
Tobias Hartmann
tobias.hartmann at oracle.com
Wed Jan 9 16:54:17 UTC 2019
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
More information about the hotspot-compiler-dev
mailing list