Camelia, the Perl 6 bug

IRC log for #moarvm, 2013-09-18

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

All times shown according to UTC.

Time Nick Message
00:49 cognominal joined #moarvm
01:02 benabik joined #moarvm
01:14 FROGGS_ joined #moarvm
02:03 dalek MoarVM/static-repr2: 934aa2e | jimmy++ | src/6model/ (3 files):
02:03 dalek MoarVM/static-repr2: added Container spec deserialization
02:03 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/934aa2ed93
02:03 dalek MoarVM/static-repr2: 8b5455a | (Gerhard R)++ | src/strings/ascii. (2 files):
02:03 dalek MoarVM/static-repr2: Add missing const to MVM_string_ascii_decode*()
02:03 dalek joined #moarvm
02:18 benabik joined #moarvm
02:21 FROGGS joined #moarvm
03:05 cognominal joined #moarvm
03:09 dalek MoarVM/static-repr2: 3e49532 | jimmy++ | src/io/procops.c:
03:09 dalek MoarVM/static-repr2: clean up MVM_proc_spawn
03:09 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/3e49532987
03:13 JimmyZ_ joined #moarvm
03:18 masak joined #moarvm
03:18 Util joined #moarvm
03:37 dalek MoarVM/static-repr2: 2eac62f | jimmy++ | src/6model/serialization.c:
03:37 dalek MoarVM/static-repr2: better error msg for 'missing serialize REPR function'
03:37 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/2eac62f2f9
05:03 diakopter TiTping
05:03 diakopter er
05:03 diakopter TimToady: ping
05:14 FROGGS joined #moarvm
05:26 TimToady pung
05:27 * TimToady is in the middle of upgrading ubunto to mint, so may come and go...
05:27 TimToady *untu
06:00 johnny5_ This is interesting: http://mailman.cs.huji.ac.il/pipermai​l/linux-il/2013-September/010649.html
06:02 johnny5_ so MoarVM could run on this soon? :D
06:03 JimmyZ only if it's posix compatibility
06:04 TimToady well, rakudo runs on the JVM, even if moarvm doesn't
06:04 TimToady and it says OSv already runs JVM stuff
06:06 johnny5_ ahh :)
06:07 TimToady but don't let me stop you from porting MoarVM to it :)
06:07 johnny5_ yeah I couldn't port my way out of a box.
06:07 johnny5_ I can barely write perl5!
06:08 johnny5_ I am progressing though.
06:08 TimToady obviously you need more input
06:08 johnny5_ haha wait is this Larry Wall? TimToady? :D
06:08 johnny5_ yeah more input.
06:10 diakopter haha port out of a box
06:13 johnny5_ Googling my suspicion I was correct.
06:13 johnny5_ It is Larry.
06:14 TimToady .oO(There goes my chance of getting any kind of honest opinion out of johnny5_...)
06:14 johnny5_ haha
06:21 johnny5_ I am a huge fan of perl and the ecosystem. The last year has been nice since technically it is my first Perl Job.
06:33 FROGGS joined #moarvm
06:51 FROGGS joined #moarvm
06:54 FROGGS o/
07:01 woolfy joined #moarvm
07:09 not_gerd joined #moarvm
07:09 not_gerd o/
07:09 yoleaux 17 Sep 2013 11:43Z <JimmyZ> not_gerd: I switched to static-repr2 branch, and removed static-repr branch
07:09 yoleaux 17 Sep 2013 14:05Z <diakopter> not_gerd: I thought previously msvc was statically linking to its microsoft .dll/.lib dependencies...? could you fix it (also if it wasn't previously, could you make it do it by default, but optional?)
07:11 not_gerd I'm going to add support for registering new reprs at runtime
07:12 not_gerd as we cannot guarantee insertion order accross instances, I'm going to get rid of the ID member of MVMREPROps
07:12 not_gerd they will still be accessible via the instance-specific registry alongside the MVMString* name
07:20 odc joined #moarvm
07:27 JimmyZ not_gerd: there are some code like if (REPR(obj)->ID == MVM_REPR_ID_MVMHash ) { ... }
07:28 not_gerd JimmyZ: for core reprs, that will still work
07:28 JimmyZ not_gerd: yeah
07:30 not_gerd I'll have to think about how to best fix that...
07:31 JimmyZ That could be great
07:32 not_gerd (some ideas: new STable entry, keeping the ID field and requiring dynamically added reprs to be unique, a hash mapping MVMREPROps* to IDs, ...)
07:32 JimmyZ I think core repr could keep the ID, but dyn repr maybe not
07:34 JimmyZ I mean core repr ID is hardcode :P
07:36 JimmyZ s/is/could be/
07:36 not_gerd on 64-bit systems, we actually can get a 32-bit STable member for free thanks to padding
07:36 not_gerd there's a 32-bit hole between size and HOW
07:43 not_gerd I'll choose the least invasive option for now (keeping the ID field in REPROps)
07:43 not_gerd it can always be refactored later
07:54 JimmyZ +1
08:34 dalek MoarVM: 2a66495 | (Tobias Leich)++ | src/mast/driver.c:
08:34 dalek MoarVM: write bytecode to disk
08:34 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/2a664959b3
08:50 jnthn I think dyn repr cna just malloc its memory :)
08:50 jnthn Then it can have different IDs in different instances.
08:50 jnthn Problem solved!
08:50 jnthn :)
08:50 jnthn But yeah, please keep ->ID field :)
08:52 not_gerd jnthn: I've actually implemented it that way
08:52 JimmyZ jnthn: How about merging static-repr2 branch?
08:52 not_gerd I'll just need to figure out why every test segfaults with my changes ;)
08:53 not_gerd I think I just forgot to root some strings
08:53 jnthn JimmyZ: um, given not_gerd++ still seems to be working on it...?
08:53 JimmyZ jnthn: that's another feature :P
08:54 JimmyZ and we can keep the branch after merging
08:54 JimmyZ Well, I just hate 'Merge Conflict' ;)
09:03 JimmyZ not_gerd: push it and maybe I can find missing root
09:04 not_gerd JimmyZ: I'm still fiddling with it
09:19 dalek MoarVM/static-repr3: 62c8b47 | (Gerhard R)++ | / (10 files):
09:19 dalek MoarVM/static-repr3: Support repr registration at runtime (still buggy)
09:19 dalek MoarVM/static-repr3: review: https://github.com/MoarVM/MoarVM/commit/62c8b47b4d
09:19 not_gerd if no one else fixes it, I'll take another look in the evening
09:19 jnthn not_gerd: This branch is based off static_repr2?
09:19 not_gerd jnthn: yes
09:19 jnthn k, thanks.
09:20 jnthn I'll try and get to the branch reviewing today :)
09:20 JimmyZ jnthn++
09:20 not_gerd it might be something really silly, but I don't have the time to look into it right now
09:20 jnthn Yeah, I got various things to attend to today also, but hopefully will get a moment later...
09:23 nwc10 the moarvm threads test still SEGVs occasionally
09:23 jnthn That may be dealt with in diakopter's gcorch branch
09:23 nwc10 ah OK
09:42 not_gerd .tell diakopter could you clarify what you want me to do re static linking - we already use /MT, or is something missing from LDLIBS?
09:42 yoleaux not_gerd: I'll pass your message to diakopter.
09:42 * not_gerd needs to run
09:42 not_gerd o/
09:42 FROGGS o/
09:42 not_gerd left #moarvm
09:42 cognominal__ joined #moarvm
10:10 woosley left #moarvm
11:59 jnap joined #moarvm
12:07 FROGGS joined #moarvm
13:03 lizmat joined #moarvm
13:36 flussence joined #moarvm
13:59 lizmat joined #moarvm
14:03 FROGGS jnthn: how do you want it factored? should bigint to the hard work or serialization.c do the hard work? https://gist.github.com/FR​OGGS/a4f263bdea31d553be84
14:05 jnthn FROGGS: serialize/deserialize, and please use the same approach that other backends do.
14:05 jnthn (which is to just turn the number into a string and write_str it)
14:05 FROGGS ahh, okay
14:05 jnthn Then we needn't leak expand_storage_if_needed
14:06 FROGGS yeah, that is why I was asking :o)
14:06 jnthn :)
14:06 jnthn yes, the "if it feels wrong it probalby is" rule often works well :)
14:06 FROGGS when I "need" to expose some stuff then already a flag raises in me internally
14:18 jnthn .tell not_gerd Looked over validation. Please document the new mark syntax in oplist (maybe just with comments at the top of the oplist file) so it's clear what they do. Other than that, +1 to merge.
14:18 yoleaux jnthn: I'll pass your message to not_gerd.
14:18 jnthn .tell not_gerd (provided tests pass, etc.)
14:18 yoleaux jnthn: I'll pass your message to not_gerd.
14:36 diakopter .
14:36 yoleaux 09:42Z <not_gerd> diakopter: could you clarify what you want me to do re static linking - we already use /MT, or is something missing from LDLIBS?
14:36 diakopter nm
14:46 dalek MoarVM: 059ad08 | (Tobias Leich)++ | src/6model/reprs/P6bigint.c:
14:46 dalek MoarVM: serialize/deserialize for bigint
14:46 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/059ad08665
14:57 jnap joined #moarvm
15:08 dalek MoarVM/static-repr2: 2a66495 | (Tobias Leich)++ | src/mast/driver.c:
15:08 dalek MoarVM/static-repr2: write bytecode to disk
15:08 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/2a664959b3
15:08 dalek MoarVM/static-repr2: 059ad08 | (Tobias Leich)++ | src/6model/reprs/P6bigint.c:
15:08 dalek MoarVM/static-repr2: serialize/deserialize for bigint
15:08 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/059ad08665
15:08 dalek MoarVM/static-repr2: e5d1bfe | jimmy++ | src/ (2 files):
15:08 dalek MoarVM/static-repr2: Merge branch 'master' into static-repr
15:08 dalek MoarVM/static-repr2:
15:08 dalek MoarVM/static-repr2: Conflicts:
15:08 dalek MoarVM/static-repr2: src/6model/reprs/P6bigint.c
15:08 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/e5d1bfe010
15:11 jnap joined #moarvm
15:15 d^_^b joined #moarvm
15:25 dalek MoarVM: 8fbec06 | jnthn++ | src/6model/reprs/P6opaque.c:
15:25 dalek MoarVM: First pass at P6opaque.serialize.
15:25 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/8fbec064ad
15:25 dalek MoarVM: 62d80c9 | jnthn++ | src/6model/reprs/MVMArray.c:
15:25 dalek MoarVM: serialize/deserialize for MVMArray.
15:25 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/62d80c93ad
15:25 dalek MoarVM: ca206be | jnthn++ | src/6model/serialization.c:
15:25 dalek MoarVM: Improve missing serialization function error.
15:25 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/ca206becf5
15:26 timotimo serialisation is the major roadblock to self-hosting compilation now?
15:26 jnthn yeah
15:27 timotimo is there lots and lots missing or only bits and pieces?
15:27 jnthn Well, major as in "needs most work"
15:27 jnthn Lots of small pieces. None of it is especially difficult.
15:27 jnthn Or at least, I hope not :)
15:27 timotimo sounds good :D
15:28 timotimo i'll tell my local .pm group "rakudo on moarvm will be ready in a couple of weeks" :P
15:29 tadzik just tell them we'll have Perl 6 ready
15:29 timotimo hehehe
15:29 dalek MoarVM: 6d587c2 | jnthn++ | src/6model/reprs/P6str.c:
15:29 dalek MoarVM: Poke serialize function into table for P6str.
15:29 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/6d587c25a0
15:30 timotimo so, it shouldn't be much work to get the threading and async io stuff up to the same point where the jvm stuff is after we get rakudo on moarvm to run 90+% spectests, yes?
15:31 jnthn It'll be work :)
15:31 jnthn But hopefully not too terrible.
15:31 timotimo easier than on parrot anyway
15:52 dalek MoarVM/static-repr2: 8fbec06 | jnthn++ | src/6model/reprs/P6opaque.c:
15:52 dalek MoarVM/static-repr2: First pass at P6opaque.serialize.
15:52 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/8fbec064ad
15:52 dalek MoarVM/static-repr2: 62d80c9 | jnthn++ | src/6model/reprs/MVMArray.c:
15:52 dalek MoarVM/static-repr2: serialize/deserialize for MVMArray.
15:52 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/62d80c93ad
15:52 dalek MoarVM/static-repr2: ca206be | jnthn++ | src/6model/serialization.c:
15:52 dalek MoarVM/static-repr2: Improve missing serialization function error.
15:52 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/ca206becf5
15:52 dalek MoarVM/static-repr2: 6d587c2 | jnthn++ | src/6model/reprs/P6str.c:
15:52 dalek MoarVM/static-repr2: Poke serialize function into table for P6str.
15:52 dalek MoarVM/static-repr2: review: https://github.com/MoarVM/MoarVM/commit/6d587c25a0
15:52 dalek MoarVM/static-repr2: 04c56fd | jimmy++ | src/6model/ (3 files):
15:52 dalek MoarVM/static-repr2: Merge branch 'master' into static-repr
15:56 timotimo dalek hung again?
15:58 benabik That'll happen with a lot of commits.
15:58 timotimo oh, i think i'm ignoring parts, joins and quits in this channel
15:58 timotimo that would explain why i saw 'em stop posting, but not quit
15:59 benabik Hm. I reenabled display of those and still don't see a dalek quit.  Eh?  Maybe an internal limit to stop those kicks.  *shrug*
16:27 JimmyZ Good night
16:28 lizmat gnight, JimmyZ!
16:35 arnsholt Don't think dalek quit
16:35 arnsholt Still listed in /names
16:35 jnthn yeah, still here
16:35 jnthn It sometimes does this too
16:36 diakopter it doesn't get kicked; it gets temp klined or whatever
16:37 benabik Kline bottled?
16:37 diakopter commutestrianizing &
16:53 foo_bar_baz joined #moarvm
17:13 jnap joined #moarvm
18:04 benabik joined #moarvm
18:24 dalek MoarVM/dynamic-repr: a8afbba | (Gerhard R)++ | / (10 files):
18:24 dalek MoarVM/dynamic-repr: Support repr registration at runtime (still buggy)
18:24 dalek MoarVM/dynamic-repr: review: https://github.com/MoarVM/MoarVM/commit/a8afbba6c6
18:24 dalek MoarVM/dynamic-repr: 5172c20 | (Gerhard R)++ | src/moarvm.c:
18:24 dalek MoarVM/dynamic-repr: Fix double-free bug
18:24 dalek MoarVM/dynamic-repr: review: https://github.com/MoarVM/MoarVM/commit/5172c20deb
18:25 not_gerd joined #moarvm
18:25 not_gerd o/
18:25 yoleaux 14:18Z <jnthn> not_gerd: Looked over validation. Please document the new mark syntax in oplist (maybe just with comments at the top of the oplist file) so it's clear what they do. Other than that, +1 to merge.
18:25 yoleaux 14:18Z <jnthn> not_gerd: (provided tests pass, etc.)
18:25 FROGGS hi not_gerd
18:30 not_gerd jnthn: static-repr2 is now known as dynamic-repr and should be fixed
18:30 jnthn not_gerd: What should I review? :)
18:31 * jnthn figures he should do this this evening, before he goes for vacation :)
18:31 not_gerd dynamic-repr should be mergeable, but isn't used yet
18:31 dalek MoarVM: 67138b1 | (Tobias Leich)++ | src/6model/reprs/KnowHOWREPR.c:
18:31 dalek MoarVM: serialize for KnowHOWREPR
18:31 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/67138b1f11
18:32 not_gerd (as in, we do not add reprs at runtime, so no guarantees that it actually works ;))
19:44 jnap joined #moarvm
20:17 flussence joined #moarvm
20:21 flussence joined #moarvm
20:44 dalek MoarVM: b764c1c | (Tobias Leich)++ | src/6model/serialization.h:
20:44 dalek MoarVM: mapped write_int32 and write_int16
20:44 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/b764c1c3e8
20:47 dalek MoarVM: a06272b | (Tobias Leich)++ | src/6model/reprs/KnowHOWAttributeREPR.c:
20:47 dalek MoarVM: added ->serialize for KnowHOWAttributeREPR
20:47 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/a06272bf11
20:51 FROGGS this is a wip of serialize_repr_data for P6opaque: https://gist.github.com/FR​OGGS/e907c786f7b18ff0c345
20:51 FROGGS it makes a few tests pass but breaks at least one
20:51 diakopter whoa
20:51 diakopter serious stuff
20:51 diakopter :D
20:51 timotimo \o/
20:52 FROGGS I need to figure out two things, num_classes and descrim
20:53 FROGGS the commented out stuff for num_classes is definitely wrong (as it explodes), the descrim stuff is just copied over as a note
20:55 not_gerd FROGGS: should I wait with merging the validation branch?
20:56 FROGGS not_gerd: no need to wait for me :o)
20:56 FROGGS my brain is already empty for today I guess
20:57 not_gerd then I'll push after a final compile/selftest
20:57 jnthn not_gerd: "+    /* FIXME: flattening necessary? */" - yes, it is before a hash lookup, so these comments in repr branch can go
20:58 jnthn *repr_static
20:58 not_gerd jnthn: just wan't sure if we needed to flatten just after creating the string
20:58 * diakopter is reminded again to debug that
21:00 jnthn not_gerd: It's probably a good idea.
21:00 jnthn I wish this branch hadn't done a bunch of MVMROOT cleanups along the way, making it harder to review...
21:03 jnthn Anyways, I like it.
21:03 jnthn Saves some allocation, some startup work, and a level of indirection on a bunch of hot paths.
21:03 FROGGS uhh, that sounds nice
21:03 jnthn So, +1 from me on merging.
21:04 jnthn not_gerd++, JimmyZ++
21:04 * timotimo listens up
21:04 dalek MoarVM: 933b810 | (Gerhard R)++ | / (8 files):
21:04 dalek MoarVM: Reimplement bytecode validation
21:04 dalek MoarVM: review: https://github.com/MoarVM/MoarVM/commit/933b8105da
21:04 dalek MoarVM: d79593b | (Gerhard R)++ | tools/update_ops.p6:
21:04 dalek MoarVM: Fix comment fail
21:05 not_gerd I expect JimmyZ will be a happy camper tomorrow ;)
21:05 diakopter why
21:05 dalek joined #moarvm
21:05 not_gerd MERGES!
21:05 jnthn 'cus he's been asking approximately 10,000 times a day for me to review those branches so we can merge them. :)
21:14 jnthn Probably not gonna be online that much in the next several days.
21:15 diakopter have a nice vacation
21:15 jnthn And may not have chance ot backlog, so .tell anything :)
21:15 jnthn Thanks :)
21:16 jnthn sleep...early flight... &
21:16 not_gerd o/
21:17 donaldh joined #moarvm
21:17 FROGGS have fun!
21:19 lizmat gnight jnthn, safe journeys!
21:21 dalek Heuristic branch merge: pushed 32 commits to MoarVM by gerdr
21:48 not_gerd good night, #moarvm
21:48 FROGGS joined #moarvm
21:49 not_gerd left #moarvm
22:22 FROGGS joined #moarvm

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