Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8165673

AArch64: Fix JNI floating point argument handling

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 9
    • Component/s: hotspot
    • Labels:
      None
    • Subcomponent:
    • Resolved In Build:
      b143
    • CPU:
      aarch64

      Description

      The jtreg test failure of

      hotspot/test/compiler/floatingpoint/Test15FloatJNIArgs.java

      is caused by incorrect floating point argument handling in JNI native wrapper.

      The following patch:

      http://people.linaro.org/~ningsheng.jian/jni-fix/webrev.00/

      can fix this bug.

      I also updated the test case (and renamed to TestFloatJNIArgs.java),
      which exposes more issues in jni floating point args handling. (Need
      to update the fake bug-id in the test java source, as I don't have
      access right to JBS). Those issues can also be fixed by the patch.

      Could someone please help to review and process the patch?

        Activity

        Show
        kvn Vladimir Kozlov added a comment - - edited Review link: http://mail.openjdk.java.net/pipermail/aarch64-port-dev/2016-September/003754.html webrev: http://cr.openjdk.java.net/~adinn/8165673/webrev.00/
        Hide
        hgupdate HG Updates added a comment -
        URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9cc82b204d20
        User: adinn
        Date: 2016-09-27 08:29:44 +0000
        Show
        hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9cc82b204d20 User: adinn Date: 2016-09-27 08:29:44 +0000
        Hide
        hgupdate HG Updates added a comment -
        URL: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/9cc82b204d20
        User: lana
        Date: 2016-11-03 02:17:58 +0000
        Show
        hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/9cc82b204d20 User: lana Date: 2016-11-03 02:17:58 +0000

          People

          • Assignee:
            adinn Andrew Dinn
            Reporter:
            aph Andrew Haley
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: