github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
Geth MoarVM: 8f6b2ac8b0 | (Zoffix Znet)++ | src/math/bigintops.c
Make big Int stringification 1.58x as fast

For Ints larger than 2⁶⁰ and only in base-10.
We do it by reducing the value by largest 10**n number that fits into a single mp_digit and doing so until our remaining number fits into a single mp_digit, at which point we continue ... (21 more lines)
00:15
MoarVM: 3740cd0bb9 | (Zoffix Znet)++ | src/math/bigintops.c
Clamp the value before we count its digits
MoarVM: 1d6562bfd6 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/math/bigintops.c
Merge pull request #984 from MoarVM/faster-Int-str

Make big Int stringification 1.58x as fast
00:35 travis-ci joined, p6bannerbot sets mode: +v travis-ci
travis-ci MoarVM build passed. Zoffix Znet 'Merge pull request #984 from MoarVM/faster-Int-str 00:35
travis-ci.org/MoarVM/MoarVM/builds/442448170 github.com/MoarVM/MoarVM/compare/9...6562bfd66a
00:35 travis-ci left 04:13 MasterDuke left 04:33 lizmat left 04:48 lizmat joined, p6bannerbot sets mode: +v lizmat 05:10 fake_space_whale left 05:49 brrt joined 05:50 p6bannerbot sets mode: +v brrt 06:00 domidumont joined, p6bannerbot sets mode: +v domidumont 06:14 brrt left 08:32 zakharyas joined, p6bannerbot sets mode: +v zakharyas 09:04 zakharyas left 09:05 zakharyas joined, zakharyas left, zakharyas joined 09:06 p6bannerbot sets mode: +v zakharyas 09:26 leont joined 09:27 p6bannerbot sets mode: +v leont
dogbert2_ brrt: I'll test your branch when I get home from $work (at work I only have a 32 bit install) 09:39
10:03 zakharyas left 10:04 zakharyas joined 10:05 p6bannerbot sets mode: +v zakharyas, domidumont left 11:03 zakharyas left 12:00 zakharyas joined 12:01 p6bannerbot sets mode: +v zakharyas 12:03 zakharyas left 12:34 robertle joined 12:35 p6bannerbot sets mode: +v robertle 13:08 zakharyas joined, p6bannerbot sets mode: +v zakharyas
dogbert2_ m: say 'ping' 13:18
camelia ping
dogbert2_ is this an old GC related problem? gist.github.com/dogbert17/c7e9b937...a957d4d62e 13:23
13:23 zakharyas left 13:25 zakharyas joined, p6bannerbot sets mode: +v zakharyas
timotimo haven't seen bind_one_param explode like that yet 13:29
dogbert2_ could it be a missing MVM_ROOT? 13:30
timotimo it's possible 13:32
does changing flags like spesh and jit and such have an impact? 13:33
dogbert2_ I'm on 32 bit so the JIT is off the hook :)
but I'm quite certain that I got it to fail at least once whit MVM_SPESH_DISABLE=1 13:34
*with
14:04 zakharyas left 14:10 zakharyas joined, p6bannerbot sets mode: +v zakharyas 14:26 ZzZombo left 14:49 greppable6 left, squashable6 left 15:31 fake_space_whale joined 15:32 p6bannerbot sets mode: +v fake_space_whale 15:36 emerson is now known as 07IAAJ8B3 16:24 greppable6 joined, squashable6 joined, niven.freenode.net sets mode: +vv greppable6 squashable6 16:25 p6bannerbot sets mode: +v greppable6, p6bannerbot sets mode: +v squashable6 16:31 greppable6 left, squashable6 left
dogbert17 brrt: trying to build MoarVM with your branch expr-jit-invoke failed for me. Got the message 'make: *** No rule to make target 'src/jit/nativecall.h', needed by 'src/main.o'. Stop.' 16:32
16:34 squashable6 joined, greppable6 joined, p6bannerbot sets mode: +v squashable6, p6bannerbot sets mode: +v greppable6
timotimo even with Configure.pl running? 16:35
if that's the case, it's probably just a line missing in Makefile.in
dogbert17 timotimo: I ran my usual 'perl Configure.pl --debug --prefix=../../install/' first 16:37
timotimo OK 16:47
then it probably misses the line in Makefile.in 16:48
17:04 zakharyas left
nine timotimo: why would the profiler show 0 % jit of a routine despite the jit log clearly showing jit compilation? 17:49
timotimo it could be inlined into the caller routine and that could be bailed by some missing op 18:10
nine: can you find mention of the routine being inlined? 18:11
nine timotimo: no, which is an oddity in itself, considering that the routine is just method (uint16 $i) { nqp::writeuint(self, nqp::elems(self), $i, 2); } 18:29
Could it be because of the manual type setup here? github.com/MoarVM/MoarVM/blob/nqp-...es.nqp#L43 18:30
japhb nine: nameless?
japhb wonders if there's something in the profiler unhappy about the nameless routine 18:31
18:48 squashable6 left, greppable6 left 18:55 AlexDaniel left 18:56 AlexDaniel joined 18:57 p6bannerbot sets mode: +v AlexDaniel 18:58 fake_space_whale left 18:59 AlexDaniel left, AlexDaniel joined 19:00 p6bannerbot sets mode: +v AlexDaniel 19:07 AlexDaniel left 19:08 AlexDaniel joined 19:09 p6bannerbot sets mode: +v AlexDaniel 19:45 greppable6 joined, squashable6 joined 19:46 p6bannerbot sets mode: +v greppable6, p6bannerbot sets mode: +v squashable6 19:59 robertle left
timotimo should be fine to have them nameless, the profiler ought to go by pointer values to identify things 20:17
22:15 07IAAJ8B3 is now known as emerson 22:38 lizmat left 22:41 acetone left 23:15 leont left 23:19 lizmat joined, p6bannerbot sets mode: +v lizmat