[cairo] Unreliable cairo-perf results; was Re: Mark cairo_fixed_t conversion functions with attribute(const)

Chris Wilson chris at chris-wilson.co.uk
Thu Mar 22 04:28:41 PDT 2007


Ross Burton (ross at burtonini.com) said: 
> You're producing some really interesting looking patches, but I don't
> see any proof that there are performance improvements.  I believe there
> are, but I like to see hard numbers!  Can you run cairo-perf-diff?

Hmm, would love to provide conclusive evidence showing that this change
improves gcc's output. However I've not found any machine where
cairo-perf gives reliable results,  e.g.

ickle at cube:~/src/cairo/perf$ git diff HEAD^ HEAD
diff --git a/src/cairo.h b/src/cairo.h
index a80efde..7701ec4 100644
--- a/src/cairo.h
+++ b/src/cairo.h
@@ -43,6 +43,8 @@
 
 CAIRO_BEGIN_DECLS
  
 +/* force a recompilation */
 +
  #define CAIRO_VERSION_ENCODE(major, minor, micro) (     \
			          ((major) * 10000)                             \
			        + ((minor) *   100)                             \
ickle at cube:~/src/cairo/perf$ ./cairo-perf-diff HEAD^ HEAD -- -i 100
Speedups
========
image-rgb     paint_image_rgba_source-256  174.08µs 0.25% -> 146.91µs 0.32%:  1.18x speedup
▏
image-rgb        paint_image_rgb_over-256  162.36µs 1.22% -> 140.51µs 0.84%:  1.16x speedup
▏
image-rgb   paint_similar_rgba_source-256  169.44µs 0.53% -> 147.17µs 0.39%:  1.15x speedup
▏
image-rgb      paint_image_rgb_source-256  160.88µs 0.79% -> 140.12µs 0.72%:  1.15x speedup
▏
...
For the record, that machine is an idle, single processor AMD64 with
1GiB of RAM.

Sorry for the whine - this has been driving me crazy.
--
Chris Wilson


More information about the cairo mailing list