<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Thanks Vladimir and Tom for the feedback,<br>
    <br>
    I agree with removing the out-of-line part.<br>
    <br>
    Removing the UsePopCountInstruction-guard in library_call also makes
    it nicer and cleaner, but noting that we in a few rare cases create
    call generators for intrinsics that always return false.<br>
    <br>
    <a
      href="http://cr.openjdk.java.net/%7Eneliasso/6658428.01/webrev.01/">http://cr.openjdk.java.net/~neliasso/6658428.01/webrev.01/</a><br>
    <br>
    Skiping reviewboard for now, doesn't seem to work well with mq and
    cygwin.<br>
    <br>
    //N<br>
    <br>
    Tom Rodriguez skrev 2012-02-21 20:05:
    <blockquote
      cite="mid:20B04FD4-5BE9-424B-927D-3BB7C075882D@oracle.com"
      type="cite">
      <pre wrap="">
On Feb 21, 2012, at 2:26 AM, Nils Eliasson wrote:

</pre>
      <blockquote type="cite">
        <pre wrap=""><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~neliasso/6658428.01/webrev.00/">http://cr.openjdk.java.net/~neliasso/6658428.01/webrev.00/</a>

6658428: C2 doesn't inline java method if corresponding intrinsic failed to inline.

Allow intrinsic inline to fallback to plain java inline case. Changing some intrinsics to not be created when they cant be used and adding some print inline changes. 
</pre>
      </blockquote>
      <pre wrap="">
I don't think you need to try inline and out of line separately.  I think the old code forced out of line because that was the only way to suppress the use of the intrinsic.  So remove the try_inline = false line and just add the extra false argument to the existing call.  It should use inline if it's appropriate or out of line if not.

In library_call.cpp I think it would be better to stop using !UsePopCountInstruction to guard those intrinsics and use the appropriate Matcher::has_match_rule for each one.  That should take into account the platform dependent meaning of UsePopCountIntstruction.

tom

</pre>
      <blockquote type="cite">
        <pre wrap="">
And I will need someone to submit it.

Thanks,
Nils E.

</pre>
      </blockquote>
      <pre wrap="">
</pre>
    </blockquote>
    <br>
    <br>
    <div class="moz-signature">-- <br>
      <a href="http://www.oracle.com" target="_blank"><img
          src="cid:part1.09090309.07090502@oracle.com" alt="Oracle"
          border="0" height="26" width="114"></a><br>
      <font color="#666666" face="Verdana, Arial, Helvetica, sans-serif"
        size="1">Nils Eliasson | Senior Member of Technical Staff<br>
        <font color="#ff0000">Oracle</font> Java Platform Group, JVM
        Engineering<br>
        ORACLE Sweden </font><br>
      <br>
      <!-- This signature was generated by the MyDesktop Oracle Business Signature utility version 3.8.8 -->
    </div>
  </body>
</html>