[parrot/parrot] 93179f: [GH #837] ChangeLog line for imcc optimizer bug

GitHub noreply at github.com
Fri Sep 21 20:51:18 UTC 2012


  Branch: refs/heads/gh837_concat
  Home:   https://github.com/parrot/parrot
  Commit: 93179fc488e50b171e3162557527cb4e259e562e
      https://github.com/parrot/parrot/commit/93179fc488e50b171e3162557527cb4e259e562e
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2012-09-21 (Fri, 21 Sep 2012)

  Changed paths:
    M ChangeLog

  Log Message:
  -----------
  [GH #837] ChangeLog line for imcc optimizer bug


  Commit: e82482e3dc7b35b3af785b89b827ed428c5c95f9
      https://github.com/parrot/parrot/commit/e82482e3dc7b35b3af785b89b827ed428c5c95f9
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2012-09-21 (Fri, 21 Sep 2012)

  Changed paths:
    A examples/benchmarks/stress_stringsu.pir

  Log Message:
  -----------
  [GH #837] Add benchmark to test encoding slowdown

stress_stringsu.pir tests encoding via sprintf and concat.
The fixed version is only a bit slower then the old broken version.
13.666s (broken) vs 13.506s (fixed) vs 3.022s (without encoding).

So fixing the encoding representation as STRING* in SREG and consts 'U'
is a worthwile goal.


  Commit: 9c8159314dd2d26365653fbcd8627b0f8fbb0559
      https://github.com/parrot/parrot/commit/9c8159314dd2d26365653fbcd8627b0f8fbb0559
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2012-09-21 (Fri, 21 Sep 2012)

  Changed paths:
    A examples/benchmarks/stress_strings1.pir

  Log Message:
  -----------
  [GH #837] New comparison benchmark against stress_stringsu.pir

Bad news:
There is almost no measurable speed advantage from ascii strings to encoded strings.
With the fix and without the fix, as sprintf handling slows down the test 4x times.

So converting a SREG and const to encoded STRING* will not gain much performance.


Compare: https://github.com/parrot/parrot/compare/7ecd47233ad1...9c8159314dd2


More information about the parrot-commits mailing list