<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 16, 2016, at 1:37 PM, Chuck Rasbold <<a href="mailto:rasbold@google.com" class="">rasbold@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Igor is correct.  The offending code is from the days when C2-only tiered compilation was the goal.</div></div></blockquote><div><br class=""></div>Excellent archeology!  The change looks good to me then.</div><div><br class=""><blockquote type="cite" class=""><div class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Thu, Jun 16, 2016 at 1:11 PM, Igor Veresov <span dir="ltr" class=""><<a href="mailto:igor.veresov@oracle.com" target="_blank" class="">igor.veresov@oracle.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="">This could be from the times when there was an attempt to implement tiered compilation using C2-only.<span class="HOEnZb"><font color="#888888" class=""><div class=""><br class=""></div></font></span><div class=""><span class="HOEnZb"><font color="#888888" class="">igor</font></span><div class=""><div class="h5"><br class=""><div class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On Jun 16, 2016, at 12:20 PM, <a href="mailto:dean.long@oracle.com" target="_blank" class="">dean.long@oracle.com</a> wrote:</div><br class=""><div class="">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000" class=""><p class="">On 6/16/16 11:55 AM, Christian Thalinger wrote:<br class="">
    </p>
    <blockquote type="cite" class="">
      
      <br class="">
      <div class="">
        <blockquote type="cite" class="">
          <div class="">On Jun 16, 2016, at 10:51 AM, Carsten Varming
            <<a href="mailto:varming@gmail.com" target="_blank" class="">varming@gmail.com</a>>
            wrote:</div>
          <br class="">
          <div class="">
            <div dir="ltr" class="">Dear compiler list,
              <div class=""><br class="">
              </div>
              <div class="">I ran into a few methods that C2 failed to
                compile due to their number of arguments. To my surprise
                C2's failure caused the methods to be executed in the
                interpreter; although they had previously been compiled
                with C1. It turns out that a few C2 failures marks a
                method as not compilable on all tiers. I went through
                all possible failures with this behavior and I don't see
                reasons for preventing C1 from compiling a method after
                C2 failed. Perhaps some of you know the reasons behind
                this. I wrote a small patch to prevent C2 failures from
                marking a method as not compilable on all tiers when
                tiered compilation is on.</div>
            </div>
          </div>
        </blockquote>
        <div class=""><br class="">
        </div>
        I briefly looked at the history of that code and it’s there
        since the initial load into Mercurial.  Does anyone have more
        history on this or is this just an oversight from early-stage
        planning?</div>
      <div class=""><br class="">
      </div>
    </blockquote>
    The _all_tiers part seems to come from
    <a href="https://bugs.openjdk.java.net/browse/JDK-4939496" target="_blank" class="">https://bugs.openjdk.java.net/browse/JDK-4939496</a>.<br class="">
    <br class="">
    dl<br class="">
    <br class="">
    <blockquote type="cite" class="">
      <div class="">
        <blockquote type="cite" class="">
          <div class="">
            <div dir="ltr" class="">
              <div class=""><br class="">
              </div>
              <div class="">I filed a bug: JDK-8159720</div>
              <div class="">and created a webrev: <a href="http://cr.openjdk.java.net/%7Ecvarming/c2_failure_prevents_all_tiers/0/" target="_blank" class=""></a><a href="http://cr.openjdk.java.net/~cvarming/c2_failure_prevents_all_tiers/0/" target="_blank" class="">http://cr.openjdk.java.net/~cvarming/c2_failure_prevents_all_tiers/0/</a></div>
              <div class=""><br class="">
              </div>
              <div class="">Carsten</div>
            </div>
          </div>
        </blockquote>
      </div>
      <br class="">
    </blockquote>
    <br class="">
  </div>

</div></blockquote></div><br class=""></div></div></div></div></div></blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></body></html>