RFR: 8216424: Remove or clean up TimeLivenessAnalysis
claes.redestad at oracle.com
Sun Jan 13 11:59:46 UTC 2019
I'll go ahead and remove it, then.
On 2019-01-13 03:20, Vladimir Kozlov wrote:
> Agree with removal. I never used it.
> 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,
>> On 09.01.19 16:10, Claes Redestad wrote:
>>> 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:
>>> .. or the flag should be removed altogether:
>>> 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.
More information about the hotspot-compiler-dev