<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=utf-8">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>Hi,</p>
    <p>webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jcm/8164508/webrev.00/">http://cr.openjdk.java.net/~jcm/8164508/webrev.00/</a></p>
    <p>bug id: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8164508">https://bugs.openjdk.java.net/browse/JDK-8164508</a></p>
    <p><br>
    </p>
    <p>we were skipping profiling of  first argument(recv)  for virtual
      call sites to static callee.  this was not done for non-inline
      case in c1. (see linked case for ref:
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8027631">https://bugs.openjdk.java.net/browse/JDK-8027631</a>)<br>
    </p>
    <pre style="color: rgb(0, 0, 0); font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span class="removed" style="color: brown;">


-      bool has_receiver = x->inlined() && !x->callee()->is_static() && !Bytecodes::has_receiver(bc);</span>
<span class="new" style="color: blue; font-weight: normal;">+      bool has_receiver = x->callee()->is_loaded() && !x->callee()->is_static() && !Bytecodes::has_receiver(bc);

<font color="#000000">above change is not absolutely necessary as this can happen only for _linkToVirtual,_linkToInterface sites inlining at present, and linker elimination and callee inlining always happen together in c1.

Please review,

Best Regards,
Jamsheed


</font></span></pre>
    <p><br>
    </p>
    <p><br>
    </p>
  </body>
</html>