[cairo] [PATCH] scaled-font: optimize cairo_scaled_font_text_to_glyphs()

Chris Wilson chris at chris-wilson.co.uk
Wed Jun 9 10:04:51 PDT 2010


On Wed,  9 Jun 2010 19:20:27 +0300, Dmitri Vorobiev <dmitri.vorobiev at movial.com> wrote:
> This patch serves two purposes. First, it factors out the heavy part
> of the cairo_scaled_font_text_to_glyphs() routine thus allowing GCC
> to better optimize the cache cleanup loop. Keeping the look-up table
> indices in a separate array speeds up array initialization even further.

The patch looks great, thanks! Care to share your benchmarks as well? A
set of random ASCII text sounds like a reasonable addition to the
microbenchmarks, but I wonder if you also have some profiles from a macro
benchmark or application that shows the effect of this change? And it
would be useful to include non-ASCII in that mix.
-ickle

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the cairo mailing list