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

Remove CompiledMethod and CodeBlobLayout classes

    XMLWordPrintable

Details

    • b18

    Description

      Undo JDK-8152664 changes [1] which was used for AOT JEP [2] implementation in JDK 9. The code was left in HotSpot assuming it will help in a future. But during work on Leyden for the past year we decided to not use it - cached compiled code will be restored in CodeCache as normal nmethods (no need to change VM's runtime and GC code to process them).

      I propose to revert JDK-8152664 change and remove CompiledMethod and CodeBlobLayout classes to simplify `nmethod` class hierarchy.

      [1] https://github.com/openjdk/jdk/commit/b853eb7f5ca24eeeda18acbb14287f706499c365
      [2] https://openjdk.org/jeps/295

      Attachments

        Issue Links

          blocks

          Enhancement - null JDK-8329629 GC interfaces should work directly against nmethod instead of CodeBlob

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

          Enhancement - null JDK-8329433 Reduce nmethod header size

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

          Enhancement - null JDK-8329628 Additional changes after JDK-8329332

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

          Enhancement - null JDK-8329629 GC interfaces should work directly against nmethod instead of CodeBlob

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

          Enhancement - null JDK-8323344 Remove unused `nmethod::_scopes_data_offset`

          • P5 - Cosmetic problem like misspelt words or misaligned text.
          • Closed

          Enhancement - null JDK-8152664 Support non-continuous CodeBlobs in HotSpot

          • P3 - Major loss of function.
          • Resolved
          links to

          Commit Commit openjdk/jdk/83eba863

          Review Review openjdk/jdk/18554

          Activity

            People

              kvn Vladimir Kozlov
              kvn Vladimir Kozlov
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: