<div dir="ltr">Hi Vladimir,<div><br></div><div>Yes, I&#39;ve followed Aleksey&#39;s post for removing UseOldInling flag.</div><div>I&#39;m just asking if this flag can be put into the obsolete_jvm_flags for JDK 8 as well. That way you won&#39;t need to add the flag back to globals.hpp.</div>
<div><br></div><div>Is it not good because you can&#39;t even issue a warning for this flag in JDK 8?</div><div><br></div><div>Thanks,</div><div>Kris</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 21, 2014 at 11:39 AM, Vladimir Kozlov <span dir="ltr">&lt;<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@oracle.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Kris,<br>
<br>
The flag name is deceiving. We only used code with UseOldInlining=true.<br>
The code which was removed was under !UseOldInlining (negated). We kept the code which was under UseOldInlining. So the behavior did not changed. And the flag is in deprecated list:<br>
<br>
+ { &quot;UseOldInlining&quot;, JDK_Version::jdk(9), JDK_Version::jdk(10) },<br>
<br>
Thanks,<br>
Vladimir<div class="im"><br>
<br>
On 2/21/14 11:15 AM, Krystal Mok wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi Vladimir,<br>
<br>
Restoring a flag without backing code is a little bit weird. Is is<br>
possible to put it in the deprecated flags list instead?<br>
<br>
Thanks,<br>
Kris<br>
<br>
<br>
On Fri, Feb 21, 2014 at 10:41 AM, Vladimir Kozlov<br></div>
&lt;<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@oracle.com</a> &lt;mailto:<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@<u></u>oracle.com</a>&gt;&gt; wrote:<br>

<br>
    <a href="https://bugs.openjdk.java.net/__browse/JDK-8035585" target="_blank">https://bugs.openjdk.java.net/<u></u>__browse/JDK-8035585</a><br>
    &lt;<a href="https://bugs.openjdk.java.net/browse/JDK-8035585" target="_blank">https://bugs.openjdk.java.<u></u>net/browse/JDK-8035585</a>&gt;<br>
    <a href="http://cr.openjdk.java.net/~__kvn/8035585/webrev/" target="_blank">http://cr.openjdk.java.net/~__<u></u>kvn/8035585/webrev/</a><div class="im"><br>
    &lt;<a href="http://cr.openjdk.java.net/~kvn/8035585/webrev/" target="_blank">http://cr.openjdk.java.net/~<u></u>kvn/8035585/webrev/</a>&gt;<br>
<br>
    I backported 8032490 changes which removed UseOldInlining flags. We<br>
    can&#39;t do that in update release.<br>
    Restore the flag but keep code changes. The removed code was never<br>
    used, tested and was rotten.<br>
<br>
    Thanks,<br>
    Vladimir<br>
<br>
<br>
</div></blockquote>
</blockquote></div><br></div>