Syntax error in perf/should_run/all.T

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Syntax error in perf/should_run/all.T

Simon Peyton Jones
That's odd. It validated for me... maybe those tests aren't run. Anyway, there was a missing paren, now fixed. Apologies

Simon

| -----Original Message-----
| From: Edward Z. Yang [mailto:ezyang at MIT.EDU]
| Sent: 18 January 2013 21:29
| To: Simon Peyton-Jones
| Cc: ghc-devs at haskell.org
| Subject: Syntax error in perf/should_run/all.T
|
| Looks like this hunk is not quite correct:
|
| commit 52234dfedb9ad07585271f33513fb2c9e5012e1a
| Author: Simon Peyton Jones <simonpj at microsoft.com>
| Date:   Thu Jan 17 13:52:35 2013 +0000
|
|     Adjust performance bounds slightly
|
| diff --git a/tests/perf/should_run/all.T b/tests/perf/should_run/all.T
| index 08ff230..cdb8852 100644
| --- a/tests/perf/should_run/all.T
| +++ b/tests/perf/should_run/all.T
| @@ -227,9 +227,10 @@ test('T5536',
|                                               1250000000)),
|                             # expected value: 1246287228 (i386/Linux)
|        if_wordsize(64,
| -          stats_num_field('bytes allocated', 2480000000,
| -                                             2510000000)),
| -                           # expected value: 2492589480 (amd64/Linux)
| +          stats_range_field('bytes allocated', 892399040, 5),
| +                           # expected value: 2,492,589,480
| (amd64/Linux)
| +                           # 17/1/13:          892,399,040
| (x86_64/Linux)
| +                           #                   (new demand analyser)
|
| The file now syntax errors for me.
|
| Edward