Re: [PATCH] Clean but not very effective optimization (lex.c / dupstring_wlen)

[Sebastian Gniazdowski <sgniazdowski@xxxxxxxxx>]

PS. I've run the F-Sy-H test via callgrind, twice, and in both runs
there is a drop in total instruction count, of 12.3 mln or 13 mln, out
of the total instruction count being 7.3 bilion. The lesser count is
of course for the optimized Zsh.
On Wed, 17 Oct 2018 at 00:08, Sebastian Gniazdowski
<sgniazdowski@xxxxxxxxx> wrote:
>
> Hello,
> dupstring_wlen (i.e. with-len) is ready to use, so this patch is just
> 4 changes dupstring(s) -> dupstring_wlen(s, l). Each of those
> dupstrings has a nice strlen() invocation already there in lex.c, a
> few lines above.
>
> Callgrind doesn't report any real change in number of instructions of
> strlen() call and a test that repeatedly invokes (z) flag on a 32-line
> buffer 150 times reports 24 ms for both unoptimized and optimized
> version.
>
> However performance test of F-Sy-Highlighting reports rather stable 40
> ms gain – out of ~3000 ms, so not much.
>
> In general simplicity of this patch made me send it.
>
> --
> Sebastian Gniazdowski
> News: https://twitter.com/ZdharmaI
> IRC: https://kiwiirc.com/client/chat.freenode.net:+6697/#zplugin
> Blog: http://zdharma.org



-- 
Sebastian Gniazdowski
News: https://twitter.com/ZdharmaI
IRC: https://kiwiirc.com/client/chat.freenode.net:+6697/#zplugin
Blog: http://zdharma.org