<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFCC">
    Looks good for the jdk files.<br>
    <br>
    Regards, Roger<br>
    <br>
    <div class="moz-cite-prefix">On 01/18/2019 10:07 AM, Gustavo Romero
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:2d4d1747-a83d-5f65-eea3-d982969ae4fd@linux.vnet.ibm.com">Hi,
      <br>
      <br>
      Could the following backport to 11u be reviewed, please?
      <br>
      <br>
      Bug     : <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8213754">https://bugs.openjdk.java.net/browse/JDK-8213754</a>
      <br>
      Change  : <a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/jdk/jdk/rev/7384e00d5860">http://hg.openjdk.java.net/jdk/jdk/rev/7384e00d5860</a>
      <br>
      Backport: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~gromero/8213754_jdk11u/v1/">http://cr.openjdk.java.net/~gromero/8213754_jdk11u/v1/</a>
      <br>
      <br>
      It adds 4 intrinsics that use instructions introduced by POWER9 in
      order to
      <br>
      speed up methods isDigit, isLowerCase, isUpperCase, and
      isWhitespace.
      <br>
      <br>
      The change is mostly PPC64-only but it does touch shared code, for
      <br>
      instance, in order to adapt the methods in question to be properly
      <br>
      intrinsified. It also needs an additional change [0], since one
      Graal
      <br>
      test has to be adapted (a separated RFR to backport [0] was sent
      to [1]).
      <br>
      <br>
      The change applies almost cleanly: only a small tweak is necessary
      because
      <br>
      the hunk for ppc.ad file relies on some absent text in the 11u
      code around
      <br>
      the change to be applied. That absent text is related to the
      Superword
      <br>
      feature (a non-related feature), which is not backported yet to
      11u.
      <br>
      <br>
      This backport was tested on POWER8 and POWER9 and no regressions
      were
      <br>
      observed.
      <br>
      <br>
      This backport was also tested on x86_64 with
      <br>
      ./test/hotspot/jtreg/compiler/{c1,c2,intrinsics} plus
      <br>
      ./test/hotspot/jtreg/compiler/graalunit (with Graal compiler
      enabled) with
      <br>
      change 8215317 [0] applied and no regressions were observed too.
      <br>
      <br>
      Thank you.
      <br>
      <br>
      Best regards,
      <br>
      Gustavo
      <br>
      <br>
      [0] <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~gromero/8215317_jdk11u/v1/">http://cr.openjdk.java.net/~gromero/8215317_jdk11u/v1/</a>
      <br>
      [1]
<a class="moz-txt-link-freetext" href="https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2019-January/032266.html">https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2019-January/032266.html</a><br>
      <br>
    </blockquote>
    <br>
  </body>
</html>