Update the expected allocations for T1969
authorIan Lynagh <igloo@earth.li>
Mon, 16 Jul 2012 19:02:09 +0000 (20:02 +0100)
committerIan Lynagh <igloo@earth.li>
Mon, 16 Jul 2012 19:02:09 +0000 (20:02 +0100)
Looking at the ticky output before and after the FastString patches,
the extra allocation seems to be in pprName, and looking at the core
it looks like it's just caused by small differences in what does and
doesn't get inlined.

The actual size of the change is small: from 578940312 to 589671096.

testsuite/tests/perf/compiler/all.T

index ff2774c..2eeda18 100644 (file)
@@ -35,12 +35,12 @@ test('T1969',
                                     #                 221667908 (x86/OS X)
                                     #                 274932264 (x86/Linux)
       if_wordsize(64,
-          compiler_stats_num_field('bytes allocated', 420000000,
-                                                      580000000)),
+          compiler_stats_range_field('bytes allocated', 589000000, 1)),
                                     # 17/11/2009:     434,845,560 (amd64/Linux)
                                     # 08/12/2009:     459,776,680 (amd64/Linux)
                                     # 17/05/2010:     519,377,728 (amd64/Linux)
                                     # 05/08/2011:     561,382,568 (amd64/OS X)
+                                    # 16/07/2012:     589,168,872 (amd64/Linux)
       only_ways(['normal']),
       extra_hc_opts('-dcore-lint')
           # Leave -dcore-lint on for this one test, so that we have something