Merge pull request #11640 from ncteisen/v1.4.x-bm-speedup-fix

V1.4.x bm_speedup fix
pull/11658/head
Noah Eisen 8 years ago committed by GitHub
commit 06543fdce7
  1. 6
      tools/profiling/microbenchmarks/bm_diff/bm_speedup.py

@ -44,13 +44,13 @@ def speedup(new, old, threshold = _DEFAULT_THRESHOLD):
s0, p0 = cmp(new, old) s0, p0 = cmp(new, old)
if math.isnan(p0): return 0 if math.isnan(p0): return 0
if s0 == 0: return 0 if s0 == 0: return 0
if p0 > _DEFAULT_THRESHOLD: return 0 if p0 > threshold: return 0
if s0 < 0: if s0 < 0:
pct = 1 pct = 1
while pct < 100: while pct < 100:
sp, pp = cmp(new, scale(old, 1 - pct / 100.0)) sp, pp = cmp(new, scale(old, 1 - pct / 100.0))
if sp > 0: break if sp > 0: break
if pp > _DEFAULT_THRESHOLD: break if pp > threshold: break
pct += 1 pct += 1
return -(pct - 1) return -(pct - 1)
else: else:
@ -58,7 +58,7 @@ def speedup(new, old, threshold = _DEFAULT_THRESHOLD):
while pct < 10000: while pct < 10000:
sp, pp = cmp(new, scale(old, 1 + pct / 100.0)) sp, pp = cmp(new, scale(old, 1 + pct / 100.0))
if sp < 0: break if sp < 0: break
if pp > _DEFAULT_THRESHOLD: break if pp > threshold: break
pct += 1 pct += 1
return pct - 1 return pct - 1

Loading…
Cancel
Save