01:14 colomon left 02:18 synopsebot joined 06:18 brrt joined 06:35 FROGGS joined
dalek arVM: 7af5df6 | raydiak++ | src/core/nativecall.c:
Allow nativecast from VMArray e.g. Blob/Buf
06:42
arVM: 04ab712 | FROGGS++ | src/core/nativecall.c:
Merge pull request #195 from raydiak/blobcast

Allow nativecast from VMArray e.g. Blob/Buf
07:23 zakharyas joined 08:13 kjs_ joined 08:49 rurban_ joined 08:53 kjs_ joined
rudi_s Could somebody please merge my patches from git://git.informatik.uni-erlangen.de/he29heri/moarvm/ - except the last one, which seems to cause build failures. 09:32
jnthn Was that the runloop label moving one? 09:33
rudi_s Yeah. 09:34
jnthn I *thought* it was in the middle rather than at the HEAD, but maybe I mis-remembered...
rudi_s jnthn: I just rebased it and reorderd it.
jnthn oh :)
rudi_s Or I could just drop it, wait a sec.
jnthn Then it's easy :)
Well, or fix it? :)
rudi_s Dropped it.
jnthn I think I mentioned something to try...
OK
:)
rudi_s Yeah, that owuld be even better. Maybe later today when I have some time.
jnthn Hm, I'm curious about "jit/graph.c: fix -Wmissing-braces warnings" 09:37
Is that 'cus it's a union?
timotimo jnthn: my int @a; my int $i; while $i < 10000000 { @a[$i] = 1; $i = $i + 1 } is many times faster than my int @a; my int $i; while $i < 10000000 { @a[$i] = 1; $i++ } 09:38
is that because of local/localref lexical/lexicalref optimization missing?
as in: $i++ has a lexref in it that means the block can't be lowered or something?
jnthn That plus nothing is smart enough to know it can inline it yet. 09:39
uh, to know how to inline it
timotimo oh
09:39 kjs_ joined
jnthn But yeah, I'm very aware of that one :) 09:39
timotimo OK :)
rudi_s jnthn: Yeah. I'm not sure why the braces are required by gcc/clang. But it causes many warnings. 09:41
jnthn ugh
Adding them is quite some clutter though.
I wonder if some compilers don't accept it without the braces... 09:42
rudi_s I find the end result not that bad, but yeah, it's not really nice.
jnthn
.oO( #pragma GCC diagnostic ignored ... )
09:44
Anyway, it builds fine and builds NQP fine 10:05
And Rakudo spectest running now 10:08
dalek arVM: 4adce22 | (Simon Ruderich)++ | src/6model/ (3 files):
remove more unused functions
10:20
arVM: 3c945be | (Simon Ruderich)++ | src/jit/graph.c:
jit/graph.c: fix -Wmissing-braces warnings
arVM: 4efe32b | (Simon Ruderich)++ | src/mast/compiler.c:
mast/compiler.c: add missing format string
MoarVM: 41b3994 | (Simon Ruderich)++ | src/core/validation.c:
MoarVM: core/validation.c: add MVM_NO_RETURN
MoarVM:
jnthn Wow, we didn't get dalek kicked for flooding :)
FROGGS hmm, but there is no review line at the end.. 10:41
jnthn Yeah, and it missed some commits too 10:42
13:22 kjs_ joined 15:04 kjs_ joined
dalek arVM/param-is-rw: 02d171c | FROGGS++ | src/ (4 files):
start to implement "is rw" params
16:16
16:46 lizmat joined 16:53 lizmat joined 17:53 kjs_ joined 19:13 dalek joined 19:43 FROGGS[mobile] joined 20:13 kjs_ joined 21:43 hoelzro joined 21:56 colomon joined 22:01 colomon left