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

[JVMCI] rationalize relationship between getCodeSize and getCode in ResolvedJavaMethod

    XMLWordPrintable

Details

    • b24

    Backports

      Description

        When ResolvedJavaMethod.getCodeSize returns a value > 0, it is currently possible for ResolvedJavaMethod.getCode to return null (i.e. when the declaring class is not linked) which has confused many JVMCI API users and led to subtle bugs. Experience has shown that a value > 0 means callers expect getCode to not return null. It's about time to conform to this reasonable expectation. If the declaring class is not linked, getCodeSize should return -1.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8297658 [JVMCI] rationalize relationship between getCodeSize and getCode in ResolvedJavaMethod

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8298359 [JVMCI] rationalize relationship between getCodeSize and getCode in ResolvedJavaMethod

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Bug - A problem which impairs or prevents the functions of the product. JDK-8297590 [TESTBUG] HotSpotResolvedJavaFieldTest does not run

            • P5 - Cosmetic problem like misspelt words or misaligned text.
            • Resolved
            links to

            Commit Commit openjdk/jdk17u-dev/b464cc2d

            Commit Commit openjdk/jdk/37848a9c

            Review Review openjdk/jdk17u-dev/934

            Review Review openjdk/jdk/11147

            Activity

              People

                dnsimon Douglas Simon
                dnsimon Douglas Simon
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: