zsh-workers
 help / color / mirror / code / Atom feed
From: Sebastian Gniazdowski <psprint@zdharma.org>
To: Bart Schaefer <schaefer@brasslantern.com>,
	zsh workers <zsh-workers@zsh.org>
Subject: Re: mikachu/badarrays (Re: Valgrind testing, ideas)
Date: Sun, 4 Jun 2017 09:08:40 +0200	[thread overview]
Message-ID: <etPan.5933b1f8.643c9869.c8d8@MacMini.local> (raw)
In-Reply-To: <170603174903.ZM16062@torch.brasslantern.com>

On 4 czerwca 2017 at 02:49:10, Bart Schaefer (schaefer@brasslantern.com) wrote:
> On Jun 2, 3:40am, Sebastian Gniazdowski wrote:
> }
> } Is there code available somewhere?
>  
> This has been pushed to the zsh git server as schaefer/badarrays

Tested it with the screensaver that has screen-buffer (24x105 in the test; pushed the code to https://github.com/zdharma/hacking-private/tree/master/array_tests).

It is 2 second speed up, from 10.15s to 8.01s. Expected more, but the numbers apparently just sum up this way, even though arrlen_le prevails in callgrind run (below). For hashes the speed is 7.91s, so it seems that hashes are already top-speed containers.

Badarrays:

3,143,167,205  itype_end
2,132,125,016  _platform_memmove$VARIANT$Nehalem [/usr/lib/system/libsystem_platform.dylib]
1,269,228,237  stringsubst
1,196,046,124  stringsubst'2
1,112,113,319  ingetc
  935,633,068  strcpy [/usr/lib/system/libsystem_c.dylib]
  928,176,451  strtod_l [/usr/lib/system/libsystem_c.dylib]
  877,921,877  dquote_parse

Normal (HEAD):

13,503,068,671  arrlen_le
 3,144,808,355  itype_end
 2,133,150,428  _platform_memmove$VARIANT$Nehalem [/usr/lib/system/libsystem_platform.dylib]
 1,487,072,023  arrlen
 1,268,544,885  stringsubst
 1,196,046,124  stringsubst'2
 1,112,113,371  ingetc
   935,739,112  strcpy [/usr/lib/system/libsystem_c.dylib]
   927,736,913  strtod_l [/usr/lib/system/libsystem_c.dylib]
   877,921,877  dquote_parse

-- 
Sebastian Gniazdowski
psprint /at/ zdharma.org


  reply	other threads:[~2017-06-04  7:08 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-26  5:04 Valgrind testing, ideas Sebastian Gniazdowski
2017-05-26  5:50 ` Bart Schaefer
2017-05-26  7:57   ` Sebastian Gniazdowski
2017-05-26 10:56     ` Daniel Shahaf
2017-05-26 23:48       ` Bart Schaefer
2017-05-28 19:43 ` mikachu/badarrays (Re: Valgrind testing, ideas) Bart Schaefer
2017-05-29 13:21   ` Mikael Magnusson
2017-05-31  5:47     ` Bart Schaefer
2017-06-01 16:31       ` Sebastian Gniazdowski
2017-06-01 20:35         ` Bart Schaefer
2017-06-02  1:40           ` Sebastian Gniazdowski
2017-06-02 22:00             ` Bart Schaefer
2017-06-04  0:49             ` Bart Schaefer
2017-06-04  7:08               ` Sebastian Gniazdowski [this message]
2017-06-01 21:22       ` Mikael Magnusson
2017-06-01 21:38         ` Bart Schaefer
2017-06-07 10:53       ` PM_ flags (Re: mikachu/badarrays ...) Oliver Kiddle
2017-06-07 18:15         ` Mikael Magnusson
2017-06-07 22:18         ` Bart Schaefer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=etPan.5933b1f8.643c9869.c8d8@MacMini.local \
    --to=psprint@zdharma.org \
    --cc=schaefer@brasslantern.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).