[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