[commit: ghc] wip/perf-testsuite: This should actually split things out this time (f81cb82)

git at git.haskell.org git at git.haskell.org
Thu Aug 10 08:44:27 UTC 2017


Repository : ssh://git@git.haskell.org/ghc

On branch  : wip/perf-testsuite
Link       : http://ghc.haskell.org/trac/ghc/changeset/f81cb82587de14333b6196e7b46d8cad93afe4b9/ghc

>---------------------------------------------------------------

commit f81cb82587de14333b6196e7b46d8cad93afe4b9
Author: Jared Weakly <jweakly at pdx.edu>
Date:   Wed Jul 26 13:52:07 2017 -0700

    This should actually split things out this time


>---------------------------------------------------------------

f81cb82587de14333b6196e7b46d8cad93afe4b9
 testsuite/driver/runtests.py | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/testsuite/driver/runtests.py b/testsuite/driver/runtests.py
index 6e4820f..62d6a8c 100644
--- a/testsuite/driver/runtests.py
+++ b/testsuite/driver/runtests.py
@@ -6,7 +6,6 @@
 
 from __future__ import print_function
 
-import argparse
 import signal
 import sys
 import os
@@ -316,15 +315,6 @@ else:
             # v-- This is in a nonsensical area. It should be happening before all of the tests are even run.
             # parse_git_notes('perf') # Should it even be happening in the test-driver logic anymore?
 
-    # This here is loading up all of the git notes into memory.
-    # It's most likely in the wrong spot and I haven't fully fleshed out
-    # where exactly I'm putting this and how I'm refactoring the performance
-    # test running logic.
-    # Currently this is useful for debugging, at least.
-    if config.use_git_notes:
-            note = subprocess.check_output(["git","notes","--ref=perf","append","-m", "\n".join(config.accumulate_metrics)])
-            parse_git_notes('perf') # Should this be hardcoded? Most likely not...
-
     if config.summary_file:
         with open(config.summary_file, 'w') as file:
             summary(t, file)



More information about the ghc-commits mailing list