Uploaded image for project: 'JikesRVM'
  1. JikesRVM
  2. RVM-537

Implement inline mustImplementInterface sequence for invokeinterface in baseline compilers

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: Compiler: Baseline
    • Labels:
      None

      Description

      Profiling indicates that a significant fraction (perhaps as high as 20%) of execution time in bloat is spent in the out-of-line routines used by the baseline compiler as part of its invokeinterface sequence.

      The inline sequence for the common case is only a few instructions, so this should have minimal impact on code space/compile time and get some nice speedups for programs that have flat execution profiles and heavy use of interfaces (ie, bloat, eclipse).

        Gliffy Diagrams

          Attachments

            Activity

            dgrove David Grove created issue -
            Hide
            dgrove David Grove added a comment -

            committed in 14522.

            Running bloat on a prototype image on ppc64-aix I see a nice speedup (1144 seconds down to 920 seconds). Unclear how much this will help with a production image, but it should have at least some impact on first iteration time (and is a step towards getting bloat on prototype/prototype-opt to run fast enough that the regression test won't time out).

            Show
            dgrove David Grove added a comment - committed in 14522. Running bloat on a prototype image on ppc64-aix I see a nice speedup (1144 seconds down to 920 seconds). Unclear how much this will help with a production image, but it should have at least some impact on first iteration time (and is a step towards getting bloat on prototype/prototype-opt to run fast enough that the regression test won't time out).
            Hide
            dgrove David Grove added a comment -

            reopening so I can modify fix target to 3.0

            Show
            dgrove David Grove added a comment - reopening so I can modify fix target to 3.0
            dgrove David Grove made changes -
            Field Original Value New Value
            Workflow jira [ 17920 ] X10 Workflow [ 19053 ]
            dgrove David Grove made changes -
            Priority Major [ 6 ] Medium [ 3 ]

              People

              • Assignee:
                dgrove David Grove
                Reporter:
                dgrove David Grove
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: