Camelia, the Perl 6 bug

IRC log for #moarvm, 2013-09-02

| Channels | #moarvm index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:06 * timotimo expects to be blown away by the performance improvements over parakudo
00:11 tadzik nqp/moar seems slightly slower than nqp/parrot now
00:12 tadzik I wonder if I'm on optimized build on either :)
00:14 diakopter parrot's optimized by default; moar is not right now
00:15 tadzik trying moar with --omg-optimized
00:15 tadzik argh. I need a thinkpad key 'S'
00:16 diakopter timotimo: it's not going to be hugely faster
00:18 tadzik on optimized build it seems about 20% faster than parrot
00:18 timotimo that sounds good :)
00:18 tadzik on nqp tests at lest
00:18 tadzik which may or may not mean somethin g:)
00:19 diakopter how much faster to run all the nqp test than nqp-jvm? :)
00:19 tadzik haha
00:20 diakopter tadzik: how are you still awake? holiday tomorrow like us usaians?
00:27 tadzik diakopter: nah, I now work tue-thu, and have a 4-day weekend
00:27 tadzik for writing my bachelor's, in theory
00:27 tadzik in practice I mostly play piano and write games
00:28 timotimo oh, games!
00:31 tadzik wanna try my asteroids? :)
00:32 tadzik I decided to be a programmer because I wanted to write games (I was 7-ish). Now I realized that I probably wrote more toy compilers than toy games
00:33 diakopter wait, compilers aren't games?
00:33 timotimo imagine perl 6 as a free to play
00:33 timotimo with microtransactions
00:34 timotimo want to buy a metaop? :D
00:34 tadzik hah. Free to play, pay to win ;)
00:34 tadzik . o O ( open source in a nutshell? )
01:06 FROGGS_ joined #moarvm
01:07 foo_bar_baz joined #moarvm
01:25 foo_bar_baz joined #moarvm
01:53 dalek MoarVM: a186fca | jimmy++ | src/core/interp.c:
01:53 dalek MoarVM: implemented compunitcodes op
01:53 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/a186fca3fa
02:07 dalek MoarVM: d92067d | jimmy++ | src/ (5 files):
02:07 dalek MoarVM: moved ContainerRegistry to instance
02:07 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/d92067dd74
03:42 dalek MoarVM: f07af83 | jimmy++ | / (7 files):
03:42 dalek MoarVM: implemented getcodecuid op
03:42 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/f07af83fe5
04:34 TimToady src/core/interp.c: In function ‘MVM_interp_run’:
04:34 TimToady src/core/interp.c:1229:21: error: duplicate case value
04:34 TimToady src/core/interp.c:1207:21: error: previously used here
04:48 TimToady case MVM_OP_compunitcodes: ended up in there twice
04:56 cognominal joined #moarvm
05:01 dalek MoarVM: 5933e8f | jimmy++ | src/core/interp.c:
05:01 dalek MoarVM: added missing getcodecuid op
05:01 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/5933e8f04d
05:54 jlaire joined #moarvm
05:54 jnthn JimmyZ: In a186fca3fa, maybe_cu and cu must be rooted, as they could become invalid after the MVM_repr_alloc_init call. But the simpler fix is just to move the line that allocates to be the first thing the op does
05:56 FROGGS joined #moarvm
05:56 JimmyZ what? jnthn, you're getting up too early
05:56 jnthn JimmyZ: Yeah...gotta commute in 10 mins or so
05:56 JimmyZ heh
05:57 jnthn Thought I'd do some code-review while I slurp my coffee and my hair dries... :)
05:57 FROGGS *g*
05:57 FROGGS morning @al
05:57 FROGGS @all
05:57 jnthn .oO( weird @all )
05:57 FROGGS and he's here in the room :o)
05:58 JimmyZ FROGGS, you're getting up too early also?
05:58 JimmyZ :P
05:58 FROGGS just a bit :o)
05:58 FROGGS that is basically the time I want to be at work
05:59 jnthn JimmyZ: Does 56-role.t pass with these updates?
05:59 JimmyZ jnthn: nope
05:59 JimmyZ Cannot locate an outer frame for the call
06:00 JimmyZ same as 44 test
06:02 jnthn ok
06:02 jnthn I won't start looking at that now, otherwise I'll forget to take the train to $dayjob :)
06:02 JimmyZ :P
06:04 dalek MoarVM: 5328f7a | jimmy++ | src/core/interp.c:
06:04 dalek MoarVM: avoids a MVMROOT
06:04 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/5328f7a630
06:04 JimmyZ jnthn: ^^ right?
06:05 jnthn JimmyZ: yes
06:06 FROGGS humm, we fail more tests than yesterday O.o
06:06 FROGGS this is new: t/nqp/44-try-catch.t .................. Could not locate compile-time value for symbol return
06:07 FROGGS ahh no, we have more test files
06:07 FROGGS t/nqp/56-role.t ....................... Cannot locate an outer frame for the call
06:07 FROGGS at nqp-src/NQP.nqp:251  (nqp.moarvm:ȷ{��*:137) <--- that $!name is weird
06:10 jnthn commute &
06:10 JimmyZ \
06:26 dalek MoarVM: d28c899 | (Tobias Leich)++ | nqp-cc/src/QASTOperationsMAST.nqp:
06:26 dalek MoarVM: import more constants from jvm-backend
06:26 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/d28c899fe0
06:36 JimmyZ FROGGS: looks like 77 test is different from nqp
06:36 FROGGS true
06:36 JimmyZ I'd like to copy 77 test to 19-file-ops.t in NQP
06:37 FROGGS yes, but it needs at least fudging for vm/parrot and/or jvm
06:37 JimmyZ oh
06:38 FROGGS or wait, there is a new file-io test in nqp since a few days, maybe you just take this one
06:38 FROGGS (committed by donaldh)
06:41 JimmyZ I avoid to commit NQP
06:43 FROGGS JimmyZ: you dont have to, but you can steal the new test file from nqp
06:45 JimmyZ we don't need to steal the new test... nqp-cc will go away
06:46 FROGGS sure
07:29 odc joined #moarvm
08:54 dalek MoarVM: 7f0dad7 | jimmy++ | src/core/interp.c:
08:54 dalek MoarVM: removed a bunch of MVMROOT
08:54 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/7f0dad76b4
09:00 donaldh joined #moarvm
09:07 dalek MoarVM: b4aaa4a | jimmy++ | src/core/interp.c:
09:07 dalek MoarVM: removed another pieces of MVMROOT
09:07 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/b4aaa4a913
09:56 foo_bar_baz joined #moarvm
10:23 dalek MoarVM: f186138 | (Tobias Leich)++ | src/io/fileops.c:
10:23 dalek MoarVM: bring nqp::readlinefh in line with jvm/parrot
10:23 dalek MoarVM:
10:23 dalek MoarVM: That is, read and return trailing newlines.
10:23 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/f186138dad
11:24 JimmyZ FROGGS: didn't you break t/nqp/77-file-io.t?
11:25 jnthn JimmyZ: 7f0dad7 was a net clarity win, but b4aaa4a makes things worse...
11:26 jnthn If we can re-order things so we naturally only access things once, go for it, but re-fetching stuff we need to refer to multiple times, then masking the smell with comments, is a bad idea.
11:31 FROGGS JimmyZ: that is because the tests are not in line with nqp@parrot and nqp@jvm
11:33 JimmyZ FROGGS: so it should contain whatever it reads CRLF?
11:34 FROGGS JimmyZ: yes, that is what nqp@parrot/jvm do, even if I'm not fond of it :o)
11:35 jnthn I fixed at least the slurp test to cope with the fact you might have \r\n
11:35 jnthn Dunno about the other tests though
11:36 FROGGS and btw, 19 would pass if CREDITS file were there
11:38 jnthn Just hack the test to look for a differnt file? Or throw a dummy CREDITS file in the repo? :)
11:38 FROGGS jnthn: I can copy it over...
11:39 JimmyZ jnthn: https://gist.github.com/zhuomingliang/6411968
11:40 JimmyZ jnthn: Is it OK?
11:42 JimmyZ since we fetch tc->instance->hll_syms only twice, 'always fetch it' is better than 'fetch it and then  MVMROOT'
11:42 jnthn JimmyZ: I don't think it is...]
11:43 jnthn We fetch hll_name twice in here too
11:43 JimmyZ oh, yes, you prefer two MVMROOT?
11:44 dalek MoarVM: 8389856 | (Tobias Leich)++ | nqp-cc/t/nqp/ (2 files):
11:44 dalek MoarVM: use CREDITS file from nqp for testing
11:44 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/8389856b4e
11:44 JimmyZ I don't like two MVMROOT though
11:44 jnthn I think it was clearer before... I'm fine if we get rid of MVMROOT for things we only fetch once just by re-ordering code, like the patch for the bigint ops
11:45 jnthn But fetching the same thing twice just to avoid using MVMROOT feels icky and will make refactoring harder.
11:45 JimmyZ jnthn: so keep it as before
11:45 JimmyZ is it OK?
11:45 jnthn Yeah. 7f0dad7 I like, but b4aaa4a I'd rather just revert.
11:46 JimmyZ well, I don't like b4aaa4a after pushing...
11:47 jnthn That's ok, version control exists to help us cope with not realizing we don't like things until later :)
11:47 JimmyZ just hate pushing and reverting it
11:47 JimmyZ *and then
11:47 jnthn yes, it's nice to realize stuff before pushing. But we're all human :)
11:49 dalek MoarVM: 9d614e0 | jimmy++ | src/core/interp.c:
11:49 dalek MoarVM: Revert "removed another pieces of MVMROOT"
11:49 dalek MoarVM:
11:49 dalek MoarVM: This reverts commit b4aaa4a913b83b96261d52f4509c2ddacf38ac54.
11:49 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/9d614e01d9
11:49 * FROGGS .oO( except jnthn *cough* )
12:00 dalek MoarVM: 549e22c | jimmy++ | src/ (2 files):
12:00 dalek MoarVM: implemented rename op, and fixed a potential read line bug
12:00 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/549e22c290
12:01 diakopter g'morn
12:02 diakopter FROGGS: yes, we already passed 3 or 4 of the 7 I copied
12:02 lizmat morn diakopter
12:02 JimmyZ diakopter: 03:00am there?
12:02 diakopter o/ :)
12:02 diakopter 5 a.m.
12:03 JimmyZ you're working behave as honeybee?
12:03 diakopter somehow caught food poisoning 1-2 days ago; slept all day yesterday when I wasn't otherwise miserable
12:05 diakopter stinks to be sick on a paid holiday ("labor" day)
12:07 diakopter at some point yesterday I hallucinated that I was working on (and finished!) a Perl 6 implementation written in the modeled world (2d dot grid) of this game Sugar Crush Saga...
12:10 woolfy Let your dreams come true as soon as you get better (I wish you get better soon).
12:10 diakopter thanks. hee-hee :)
12:11 woolfy oO(Sugar Crush, that's a game, hmm, never saw it, just read about it)
12:13 diakopter it's very cute. reminds me of the movie Wreck It Ralph, which is one of the best movies *ever* (it was released this year)... I *highly* recommend watching it.  very amusing and touching.
12:14 woolfy oO(wha? huh?)
12:15 woolfy http://www.imdb.com/title/tt1772341/   (has a rating of 7.8 which does not indicate it is one of the best movies ever)(is a movie from 2012, which is not this year)
12:16 woolfy diakopter, go back to bed!  :-)
12:18 JimmyZ could anyone help me download http://www.youtube.com/watch?v=TIUUVEojULE ?
12:19 lizmat woolfy: go back to bed, you're not feeling well either!
12:19 JimmyZ I nerver can visit youtube in china
12:19 woolfy ok ok bye
12:21 moritz JimmyZ: http://moritz.faui2k3.org/tmp/jnthn-moarvm.webm
12:21 donaldh joined #moarvm
12:22 JimmyZ moritz: thanks very much
12:47 jnthn .oO( wow, really effective firewall there, China :P )
12:47 jnthn diakopter: Get well soon.
12:58 timotimo no firewall can handle aggressively adaptive protocols (like humans)
13:05 Woodi joined #moarvm
13:05 Woodi hallo :)
13:05 lizmat Woodi  o/
13:12 dalek MoarVM: fd6faee | (Tobias Leich)++ | nqp-cc/t/nqp/ (4 files):
13:12 dalek MoarVM: merge 77-file-io.t into 19-file-ops.t
13:12 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/fd6faeede9
13:36 FROGGS joined #moarvm
14:28 donaldh joined #moarvm
14:48 FROGGS joined #moarvm
16:15 diakopter woolfy: it's a good movie :P
16:16 woolfy diakopter: the description suggests so, I just never heard of it. I will add it to my want-list
16:18 woolfy Liz is a bit stunned: how did diakopter know that woolfy came out of bed...  :-)
16:23 * diakopter just woke up, again
16:24 diakopter I have a feeling I'll wake up again in another 4 hours
16:28 * woolfy too
17:24 FROGGS joined #moarvm
18:22 * diakopter might be getting hungry, finally
18:23 FROGGS that's a good sign
18:28 timotimo i hope you'll recover soon!
18:44 jnthn joined #moarvm
18:49 diakopter jnthn: the 24-module.t failure relates to the prior_invocation thingie
18:52 jnthn diakopter: Yeah. prior_invocation must die, though.
18:53 diakopter what was it bandaiding?
18:53 diakopter I can try to fix it if you tell me how
18:59 jnthn diakopter: It was before I talked with Pm at YAPC::NA about closures :)
18:59 jnthn I think it's one I need to work out; I'm not completely sure I can provide instructions that'll work, and also I want to fix this stuff on JVM too.
19:00 diakopter oh
19:21 * masak .oO( priori invocati )
19:22 * FROGGS .oO( whispered masak with a flick of his wand )
19:23 sorear .oO( masak is porting Perligata? )
19:50 BabsSeed joined #moarvm
20:29 jnthn hm
20:29 jnthn Non-zero exit status: 5
20:29 jnthn On 49-regex-interpolation.t
20:29 jnthn We haz a segfaulter...
20:29 benabik 5 == segfault?
20:29 jnthn yeah
20:30 jnthn (Note, Windows :))
20:30 * benabik shudders.
20:30 benabik ;-)
20:33 jnthn Urgh. Memory corruption.
20:34 timotimo damn politicians everywhere
20:34 jnthn Anything with a thread ID of 13 in a single-threaded program is le hosed
20:34 timotimo identity crisis?
20:35 jnthn It's st is null too
20:35 jnthn *Its
20:35 jnthn 0x0000000000b12b20
20:35 jnthn uh, sorry
20:38 jnthn Hm, yes, object pointer is into fromspace...
20:38 jnthn That's always wrong.
20:39 timotimo fromspace is the "take from here" space of the GC?
20:40 jnthn It's the semi-space we were allocating in before the last GC.
20:40 jnthn And that we evacuated all living things from during the last GC
20:49 sorear haha 13
20:50 jnthn unlucky, huh? :P
20:50 sorear nm was confusing 13 (SIGPIPE) with 11 (SIGSEGV)
20:56 dalek MoarVM: fabda86 | jnthn++ | src/gc/debug.h:
20:56 dalek MoarVM: Add header with handy GC debugging macro.
20:56 dalek MoarVM:
20:56 dalek MoarVM: Have hacked this into code a couple of times while debugging GC issues
20:56 dalek MoarVM: and figured it's best shared and easily available.
20:57 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/fabda86d29
21:00 jnthn static MVMObject *env_hash;
21:00 jnthn if (!env_hash) {
21:00 jnthn argh!!
21:00 timotimo oops :)
21:00 jnthn And not only is it static
21:00 jnthn It never gets permrooted.
21:00 jnthn Meaning it gets GC'd.
21:00 jnthn So some time later, env_hash points into heck knows what.
21:02 * jnthn is learning that the "is it in fromspace" sanity test is one of the unforseen debugging advantages of having chosen a semi-space collector for the nursery...
21:10 dalek MoarVM: 94ade18 | jnthn++ | src/ (3 files):
21:10 dalek MoarVM: Hang env_hash off instance, mark it.
21:10 dalek MoarVM:
21:10 dalek MoarVM: Fixes a segfault that 49-regex-interpolation.t triggered, but really
21:10 dalek MoarVM: anything unlucky enough could have.
21:10 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/94ade18efa
21:14 jnthn Well, there's my patch for today
21:34 * lizmat is considering zapping tests that do chop(@array) and chop(%hash)
21:34 lizmat tests originate from 2008, aka 5+ years ago, and seem to have no bearing on reality or the future
21:35 lizmat t/spec/S32-str/chop.t for the ones interested
21:36 lizmat oops ww
22:56 diakopter hi
23:04 cognominal joined #moarvm
23:13 benabik joined #moarvm
23:24 foo_bar_baz joined #moarvm

| Channels | #moarvm index | Today | | Search | Google Search | Plain-Text | summary