<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <br>

    <br>

    <div class="moz-cite-prefix">On 10/26/17 10:03 AM, Paul Sandoz

      wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:A3420FD7-0556-4865-9CE7-2E5252DD1381@oracle.com">

      <pre wrap="">Hi,

Please review the following patch for minimal dynamic constant support:

  <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8186046-minimal-condy-support-hs/webrev/">http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8186046-minimal-condy-support-hs/webrev/</a> <a class="moz-txt-link-rfc2396E" href="http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8186046-minimal-condy-support-hs/webrev/"><http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8186046-minimal-condy-support-hs/webrev/></a>

</pre>

    </blockquote>

    <br>

    <br>

    I reviewed the non-hotspot change as a learning exercise (I am not

    close to j.l.invoke implementation).Â  I assume DynamicConstant

    intends to be non-public in this patch, right?<br>

    <pre>  30 public final class DynamicConstant 

Mandy

</pre>

  </body>

</html>