RVM
  1. RVM
  2. RVM-584

Automate calculation of compilerDNA from compiler-dna test run Report.xml

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1.4
    • Component/s: Infrastructure: Build
    • Labels:
      None
    • Number of attachments :
      0

      Description

      There are a few manual steps involved in taking the data in the Report.xml file produced by the compiler-dna test run an getting the final numbers to plug into CompilerDNA.java. They are documented on this wiki page: http://docs.codehaus.org/display/RVM/Compiler+DNA

      We do this fairly infrequently, but it would be nice to extend the compiler-dna test run so that the compilationRates and speedupRates arrays could just be copied directly from a field in the Report.xml file. I suspect that for someone who was better at ant than me, this would be a really easy thing to do.

        Activity

        Hide
        Ian Rogers added a comment -

        It would also be nice to watch these values as a performance graph.. this way we can see when we need to update the compiler dna following changes to the RVM.

        Show
        Ian Rogers added a comment - It would also be nice to watch these values as a performance graph.. this way we can see when we need to update the compiler dna following changes to the RVM.
        Hide
        David Grove added a comment -

        I agree with Ian's comment in principle, but gathering the DNA takes a long time. 2-3 hours on a fast IA32 machine and 3-4 hours on a fast PPC machine. Maybe a weekly job given the machine resources we have?

        Show
        David Grove added a comment - I agree with Ian's comment in principle, but gathering the DNA takes a long time. 2-3 hours on a fast IA32 machine and 3-4 hours on a fast PPC machine. Maybe a weekly job given the machine resources we have?
        Hide
        Ian Rogers added a comment -

        This issue is fixed in MRP-18

        Show
        Ian Rogers added a comment - This issue is fixed in MRP-18
        Hide
        Erik Brangs added a comment -

        I'll pull the fix from MRP.

        Show
        Erik Brangs added a comment - I'll pull the fix from MRP.
        Hide
        Erik Brangs added a comment -

        Pulled fix from MRP in a987aeb1642fdd74316ad41043b15c8be2f305ec.

        Show
        Erik Brangs added a comment - Pulled fix from MRP in a987aeb1642fdd74316ad41043b15c8be2f305ec .

          People

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

            Dates

            • Created:
              Updated:
              Resolved: