<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    I suggest doing the tiers 1-4 testing separate from the tier5+
    testing, to reduce noise.<br>
    <br>
    There is a fix for CheckGraalIntrinsics coming to upstream Graal. 
    When Jesper merges the overwritten changes, he could include that
    fix as well, so that <a role="listbox" class="ui basic compact
      dropdown" tabindex="0"><span>compiler/graalunit/HotspotTest.java
        passes.<br>
        <br>
        The HeapMonitorStatArrayCorrectnessTest failure should have been
        fixed by JDK-8223441, unless Jesper's test repo is out of date.<br>
        <br>
        dl<br>
      </span></a><br>
    <div class="moz-cite-prefix">On 5/14/19 12:32 PM, Vladimir Kozlov
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:a5b07b39-6725-3e3a-b276-1dee6465fd57@oracle.com">Changes
      seems fine but I am not comfortable about tests results. There are
      a lot of timeouts again but there are many graalunit tests
      failures.
      <br>
      <br>
      This time you have to apply overwritten diffs after merge (if we
      decide to push it) - these changes are not in Graal master repo
      yet.
      <br>
      <br>
      Thanks,
      <br>
      Vladimir
      <br>
      <br>
      On 5/13/19 5:19 PM, <a class="moz-txt-link-abbreviated" href="mailto:jesper.wilhelmsson@oracle.com">jesper.wilhelmsson@oracle.com</a> wrote:
      <br>
      <blockquote type="cite">Hi,
        <br>
        <br>
        Please review the patch to integrate recent Graal changes into
        OpenJDK.
        <br>
        Graal tip to integrate: 6a18d9ddacd8eecb0ae4877f687e171889939c0d
        <br>
        <br>
        Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8223346">https://bugs.openjdk.java.net/browse/JDK-8223346</a>
        <br>
        Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jwilhelm/8223346/webrev.00/">http://cr.openjdk.java.net/~jwilhelm/8223346/webrev.00/</a>
        <br>
        <br>
        This integration did overwrite changes already in place in
        OpenJDK. The diff has been attached to the umbrella bug.
        <br>
        <br>
        Thanks,
        <br>
        /Jesper
        <br>
        <br>
      </blockquote>
    </blockquote>
    <br>
  </body>
</html>