Hi Andy, > commit 24ea9f9c3abb8d9398df4810b815075593ba67c8 > Author: Andy Wingo <wingo-e+axbwqsrlaavxtiumw...@public.gmane.org> > Date: Tue Mar 6 22:21:39 2012 +0100 > > ports.c: inline get_codepoint > > * libguile/ports.c (get_codepoint): Add inline keyword. It showed up > high in benchmarks, and it's static, so it's probably important to > inline.
It's generally frowned upon nowadays to explicitly ask the C compiler to inline functions, as modern compilers usually do a better job of deciding when to inline than we do. It's not a very small function, and now there will be two copies of it, maybe more in the future. The overhead of calling a static function is extremely low. Were you able to measure how much this helps performance? Thanks, Mark