<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="">+            Asserts.assertTrue(ids.add(wbCompileID), testCase<br class="">+                    + " : vm compilation allocated existed id " + id);<br class="">+            Asserts.assertTrue(ids.add(id), testCase<br class="">+                    + " : allocateCompileId returned existed id " + id);<br class=""><br class="">I think these should say “existing id”.  Otherwise this looks good.<div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Nov 10, 2015, at 8:26 AM, Tatiana Pivovarova <<a href="mailto:tatiana.pivovarova@oracle.com" class="">tatiana.pivovarova@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  

    <meta http-equiv="content-type" content="text/html; charset=utf-8" class="">
  
  <div text="#000000" bgcolor="#FFFFFF" class="">
    <font face="Helvetica, Arial, sans-serif" class="">Hy guys!<br class="">
      <br class="">
      Please review this small patch<br class="">
      I just added loop to check that the allocation allocate new id on
      each iteration and used length of bytecode to define incorrect
      bci.<br class="">
      <br class="">
      Test:<br class="">
        jtreg<br class="">
      <br class="">
      webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tpivovarova/8138810/webrev.00/">http://cr.openjdk.java.net/~tpivovarova/8138810/webrev.00/</a><br class="">
      bugid: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8138810">https://bugs.openjdk.java.net/browse/JDK-8138810</a><br class="">
      <br class="">
      Thanks,<br class="">
      Tatiana<br class="">
    </font>
  </div>

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