<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Thank you!<br>
    <blockquote
      cite="mid:6B4C60B3-15C2-4F4C-BA86-42BEA270C2B1@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <br class="">
      <div>
        <blockquote type="cite" class="">
          <div class="">On Nov 5, 2015, at 7:32 AM, Dmitrij Pochepko
            <<a moz-do-not-send="true"
              href="mailto:dmitrij.pochepko@oracle.com" class="">dmitrij.pochepko@oracle.com</a>>
            wrote:</div>
          <br class="Apple-interchange-newline">
          <div class="">
            <meta http-equiv="content-type" content="text/html;
              charset=windows-1252" class="">
            <div text="#000000" bgcolor="#FFFFFF" class=""> Hi,<br
                class="">
              <br class="">
              please review fix for:<br class="">
              <a moz-do-not-send="true" class="moz-txt-link-freetext"
                href="https://bugs.openjdk.java.net/browse/JDK-8141552">https://bugs.openjdk.java.net/browse/JDK-8141552</a>
              - [TESTBUG]
              compiler/jvmci/events/JvmciNotifyInstallEventTest failed
              after jvmci refresh<br class="">
              <br class="">
              A problem was that test tried to install same
              CompilationResult code twice, which is not allowed because
              HotSpotCodeCacheProvider.installCode method sets
              finalizeLayout state for DataSection inside
              CompilationResult and this state can't be set twice for
              same instance(there is an assertion verifying that). So,
              as a fix, i've added creation and initialization of
              another instance to have a new instance with fresh state.<br
                class="">
              <br class="">
              webrev: <a moz-do-not-send="true"
                href="http://cr.openjdk.java.net/%7Edpochepk/8141552/webrev.01/"
                class="">http://cr.openjdk.java.net/~dpochepk/8141552/webrev.01/</a><br
                class="">
            </div>
          </div>
        </blockquote>
        <div><br class="">
        </div>
        Looks good.</div>
      <div><br class="">
        <blockquote type="cite" class="">
          <div class="">
            <div text="#000000" bgcolor="#FFFFFF" class=""> <br
                class="">
              I've tested it on linux-amd64.<br class="">
              <br class="">
              Thanks,<br class="">
              Dmitrij<br class="">
              <meta http-equiv="content-type" content="text/html;
                charset=windows-1252" class="">
            </div>
          </div>
        </blockquote>
      </div>
      <br class="">
    </blockquote>
    <br>
  </body>
</html>