01:15 Kaiepi joined 02:56 ilbot3 joined 05:23 AlexDaniel joined 05:49 releasable6 joined 06:54 greppable6 joined 07:06 bisectable6 joined, nativecallable6 joined, bisectable6 joined 07:27 robertle joined 07:54 bartolin joined 08:05 committable6 joined 08:45 quotable6 joined 09:37 shareable6 joined, unicodable6 joined 09:39 releasable6 joined 09:51 domidumont joined 09:57 domidumont joined 10:34 bisectable6 joined, benchable6 joined, committable6 joined 10:35 statisfiable6 joined 12:26 Kaiepi joined 15:02 bart joined
brrt good * 15:06
i can't replicate the failure of MVM_box_str
on my machine :-(
MasterDuke have you tried a linux vm? you're on osx, correct? 15:10
would it help if i did a jit-bisect again? 15:12
brrt i'm on fedora 15:13
specially for this
:-( 15:18
you'll never hear me complain about bugs, but bugs i can't replicate are... madenning
*maddening
MasterDuke: if you would? 15:19
maybe only in combination with box_i
since i haven't enabled that one
i can replicate them when they're both enabled 15:28
(and it's bisectable. this is going to be weird) 15:32
15:36 committable6 joined
brrt also, bugs in CORE.setting compilation suffer from being rather slow to bisect 15:39
MasterDuke bisect with both box_s and box_i? 15:48
afk for a while, but i started it running 15:52
16:01 brrt joined 16:04 zakharyas joined 16:24 zakharyas joined, Kaiepi joined 16:31 Kaiepi joined
timotimo we cache the envhash even though getenvhash is marked "usehll" 16:38
16:43 unicodable6 joined 17:09 Kaiepi joined
timotimo rather surprised to see that getenvhash is hardly encountered by spesh at all 17:38
perhaps the code that uses getenvhash should be changed to access it via nqp::getenvhash more often rather than binding to a variable 17:39
trying to get note("foobar") if nqp::getenvhash<LOUD> to disappear if LOUD is not set in the environment 18:01
there's so many left-overs %)
MasterDuke .tell brrt the box_s jit-bisect data is here gist.github.com/MasterDuke17/e74e1...256ba2487a 18:03
yoleaux MasterDuke: I'll pass your message to brrt.
MasterDuke timotimo: what do you mean left-overs? 18:06
timotimo bytecode that was involved in the check and call 18:18
MasterDuke ah 18:19
timotimo why do we have PHI nodes with only two arguments %) 18:20
YES! \o/ 18:22
all that's left from this whole ordeal is the const_s for the LOUD string
it's probably only left because of deopt boundaries 18:23
hm, the effect isn't quite as extreme as i'd hoped 18:26
7.7s before my patch, 6.9s after my patch
perl6 -e 'use nqp; for ^5_000_000 { say "test" if nqp::getenvhash<LOUD> };'
though with the debug output completely removed from the bytecode, i imagine it could press a frame or two below the inline limit 18:27
MasterDuke cool 18:29
timotimo we just need to rewrite every usage of debug switching in nqp and rakudo to use this pattern instead of assigning to a variable of some kind 18:30
because variable assignment could very well mess this up
(more tests surely needed) 18:34
unfortunately, my code breaks nqp's build rather early
19:57 hoelzro joined 20:34 zakharyas joined 20:46 zakharyas joined 21:19 Kaiepi joined 21:25 Kaiepi joined 21:32 Kaiepi joined 21:34 Kaiepi joined 21:37 Kaiepi joined
MasterDuke timotimo: btw, have you ever looked at www.libtom.net/TomsFastMath/ ? 21:44
timotimo only briefly peeked at the main page 21:48
MasterDuke think there's any chance moarvm could use (at least some of) it? 21:50
timotimo probably 22:00
23:15 Kaypie joined
Geth MoarVM: Kaiepi++ created pull request #810:
Panic when Dyncall fails to allocate native callbacks
23:27