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@MIT.EDU]
| Sent: 18 January 2013 21:29
| To: Simon Peyton-Jones
| Cc: ghc-devs@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
| 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
