<div dir="ltr">Hi Vladimir,<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 21, 2014 at 12:02 PM, 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 2/21/14 11:52 AM, Krystal Mok wrote:<br>

</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hi Vladimir,<br>
<br><div class="im">
Yes, I&#39;ve followed Aleksey&#39;s post for removing UseOldInling flag.<br>
I&#39;m just asking if this flag can be put into the obsolete_jvm_flags for<br>
JDK 8 as well. That way you won&#39;t need to add the flag back to globals.hpp.<br>
<br>
Is it not good because you can&#39;t even issue a warning for this flag in<br>
JDK 8?<br>
</div></blockquote>
<br>
Not in Update release. We are talking about 8u20.<br>
<br></blockquote><div>I see. Then this change looks good to me (not a Reviewer).</div><div><br></div><div>There are counter-examples to deprecating flags in a update release, though, e.g.</div><div>  { &quot;Tier1Inline&quot;,         JDK_Version::jdk_update(7,40), JDK_Version::jdk(8) },<br>
</div><div>I just though it&#39;d be doable for JDK_Version::jdk_update(8, 20) as well...</div><div><br></div><div>Thanks,</div><div>Kris</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Thanks,<br>
Vladimir<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
<br>
Thanks,<br>
Kris<br>
<br>
<br>
On Fri, Feb 21, 2014 at 11:39 AM, Vladimir Kozlov<br></div><div class="im">
&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>
    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<br>
    kept the code which was under UseOldInlining. So the behavior did<br>
    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<br>
<br>
<br>
    On 2/21/14 11:15 AM, Krystal Mok wrote:<br>
<br>
        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>
        &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;<br></div>

        &lt;mailto:<a href="mailto:vladimir.kozlov@" target="_blank">vladimir.kozlov@</a>__<a href="http://oracle.com" target="_blank">orac<u></u>le.com</a><br>
        &lt;mailto:<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@<u></u>oracle.com</a>&gt;&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>
             &lt;<a href="https://bugs.openjdk.java." target="_blank">https://bugs.openjdk.java.</a>__<u></u>net/browse/JDK-8035585<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;&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><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;<div class="im"><br>
<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><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;&gt;<br>
<br>
             I backported 8032490 changes which removed UseOldInlining<br>
        flags. We<br>
             can&#39;t do that in update release.<br>
             Restore the flag but keep code changes. The removed code<br>
        was never<br>
             used, tested and was rotten.<br>
<br>
             Thanks,<br>
             Vladimir<br>
<br>
<br>
<br>
</div></blockquote>
</blockquote></div><br></div></div>