[LNT] r239032 - Fix a pre-coffee think-o in r238031

James Molloy james.molloy at arm.com
Thu Jun 4 03:04:22 PDT 2015


Author: jamesm
Date: Thu Jun  4 05:04:21 2015
New Revision: 239032

URL: http://llvm.org/viewvc/llvm-project?rev=239032&view=rev
Log:
Fix a pre-coffee think-o in r238031

...The comment was right but the if condition wasn't!

Modified:
    lnt/trunk/lnt/server/reporting/analysis.py

Modified: lnt/trunk/lnt/server/reporting/analysis.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/reporting/analysis.py?rev=239032&r1=239031&r2=239032&view=diff
==============================================================================
--- lnt/trunk/lnt/server/reporting/analysis.py (original)
+++ lnt/trunk/lnt/server/reporting/analysis.py Thu Jun  4 05:04:21 2015
@@ -55,7 +55,7 @@ class ComparisonResult:
 
         # Special case: if we're using the minimum to aggregate, swap it for max
         # if bigger_is_better.
-        if aggregation_fn == stats.safe_min:
+        if aggregation_fn == stats.safe_min and bigger_is_better:
             aggregation_fn = stats.safe_max
         
         if samples:





More information about the llvm-commits mailing list