]> git.wincent.com - wikitext.git/commit
Let the compiler decide when to inline
authorWincent Colaiuta <win@wincent.com>
Fri, 29 Feb 2008 12:12:23 +0000 (13:12 +0100)
committerWincent Colaiuta <win@wincent.com>
Fri, 29 Feb 2008 12:12:23 +0000 (13:12 +0100)
commitcf1128cd9064668c44d61060a42be37fac8d4dff
tree7daf0b1e151f2fd6f9d43847713d323b414e9307
parent44d9cec8b530dfd0f756c98025b5987a1f46e750
Let the compiler decide when to inline

This change was initially prompted because I was getting duplicate
symbol errors when linking for functions which should have been
inlined on RHEL 5.1.

My first trouble-shooting technique was disabling inlining to confirm
that the linker errors went away (they did) but it turns out that there
was a secondary benefit: the compiler is actually much smarter than me,
because when I leave things up to the compiler instead of explicitly
requesting inlining, the spec suite runs 10 to 20% faster. There is
quite a bit of Ruby/RSpec overhead in the spec suite, so the speed
improvement of the parser itself is probably significantly more.

Signed-off-by: Wincent Colaiuta <win@wincent.com>
ext/ary.c
ext/ary.h
ext/parser.c
ext/str.c
ext/str.h