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