Perl 6 - the future is here, just unevenly distributed

IRC log for #moarvm, 2017-11-30

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

All times shown according to UTC.

Time Nick Message
00:00 bisectable6 joined #moarvm
00:00 committable6 joined #moarvm
00:29 evalable6 joined #moarvm
00:30 bisectable6 joined #moarvm
00:30 committable6 joined #moarvm
00:59 evalable6 joined #moarvm
01:00 bisectable6 joined #moarvm
01:00 committable6 joined #moarvm
01:13 committable6 joined #moarvm
01:43 xi- joined #moarvm
02:03 xi- joined #moarvm
02:18 zakharyas joined #moarvm
02:59 ilbot3 joined #moarvm
02:59 Topic for #moarvm is now https://github.com/moarvm/moarvm | IRC logs at  http://irclog.perlgeek.de/moarvm/today
06:39 brrt joined #moarvm
06:42 Geth ¦ MoarVM: briandfoy++ created pull request #759: Update Unicode version
06:42 Geth ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/pull/759
06:46 brrt good * #moarvm
06:53 domidumont joined #moarvm
06:59 domidumont joined #moarvm
07:08 Geth ¦ MoarVM: deddb916cd | (brian d foy)++ (committed by Samantha McVey) | README.markdown
07:08 Geth ¦ MoarVM: Update Unicode version cited in README
07:08 Geth ¦ MoarVM:
07:08 Geth ¦ MoarVM: The moarvm.org page touts Unicode 10, so update the README to be
07:08 Geth ¦ MoarVM: consistent.
07:08 Geth ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/commit/deddb916cd
07:08 Geth ¦ MoarVM: 0406d43143 | (Samantha McVey)++ (committed using GitHub Web editor) | README.markdown
07:08 Geth ¦ MoarVM: Merge pull request #759 from briandfoy/patch-1
07:08 Geth ¦ MoarVM:
07:08 Geth ¦ MoarVM: Update Unicode version cited in README
07:08 Geth ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/commit/0406d43143
07:08 geospeck joined #moarvm
07:09 samcv Never taken advantage of the fact i can edit peoples commits before merge. I think i'll do that more often to adjust wording in commit titles
07:10 domidumont joined #moarvm
07:50 brrt joined #moarvm
09:18 Voldenet joined #moarvm
09:18 Voldenet joined #moarvm
09:33 geospeck joined #moarvm
09:40 zakharyas joined #moarvm
09:40 BinGOs joined #moarvm
09:40 BooK joined #moarvm
09:42 domidumont joined #moarvm
09:57 domidumont joined #moarvm
10:12 mtj_ joined #moarvm
10:14 dogbert2 o/
10:14 * dogbert2 wonders if jnthn has any brane today :)
10:15 Geth joined #moarvm
10:15 SourceBaby joined #moarvm
10:15 synopsebot joined #moarvm
10:15 dalek joined #moarvm
10:38 * jnthn also :)
10:39 nwc10 well, we have snow
10:39 nwc10 no idea about braaaaaaaaaaaanes
10:40 jnthn Darn, the snow missed here.
10:40 dogbert2 the snow where I'm at melted away
10:48 ilmari it was down to freezing here overnight, but no snow, and it's up to 3°C now
10:49 * jnthn works on a Cro middleware task as a warm-up :)
11:13 jnthn Well, that went to plan :)
11:17 dogbert2 an easy fix then ?
11:20 jnthn Was actually implementing a feature, for once, rather than fixing something ;)
11:21 dogbert2 cool
11:29 brrt good *, btw
11:30 dogbert2 hello brrt
11:30 brrt i note that i, for one, do not have any brane left
11:30 brrt they were all out this morning
11:30 dogbert2 perhaps jnthn stole your brane :)
11:31 brrt i blame the lack of snow
11:32 jnthn Grr, this inlines/handlers thing is going to be quite tricky
11:41 yoleaux joined #moarvm
11:43 travis-ci joined #moarvm
11:43 travis-ci MoarVM build errored. Samantha McVey 'Merge pull request #759 from briandfoy/patch-1
11:43 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/309367478 https://github.com/MoarVM/MoarVM/compare/d23f5ca1635c...0406d431437b
11:43 travis-ci left #moarvm
11:46 jnthn Really so. Just discovering the number of places that assume a flat table.
11:49 jnthn And noting that each of them will get more complex.
11:52 jnthn Think I'll see if I can find an alternate solution that's going to be less invasive, now I've seen the cost of my original idea
12:02 yoleaux joined #moarvm
12:08 jnthn lunch &
12:29 AlexDaniel` joined #moarvm
12:50 zakharyas joined #moarvm
12:54 harrow joined #moarvm
13:01 ilmari[m] joined #moarvm
13:02 lizmat joined #moarvm
13:15 lizmat joined #moarvm
13:16 jnthn om nom borsch
13:16 jnthn So I think I have a far easier to implement alternative solution
13:16 nwc10 \o/
13:16 jnthn We poke entries into the handlers extent list that mark the boundaries of an inline
13:17 jnthn Thus the handler walker knows when it's crossing such a boundary
13:18 jnthn That provides a way forward on the "skip the caller, then look for outer" issue
13:19 jnthn If that extent list entry were to also contain an indication of which inline we just crossed the boundary of, then we can use that to do lookups to decide if this is or isn't a decent outer
13:24 jnthn src/strings/unicode.c: In function ‘MVM_unicode_get_property_str’:
13:24 jnthn src/strings/unicode.c:72236:83: warning: pointer/integer type mismatch in conditional expression ? Bidi_Mirroring_Glyph_enums[0] : Bidi_Mirroring_Glyph_enums[result_val]) : bogus;
13:24 jnthn samcv: ^
13:48 brrt jnthn, i'm not sure if i understood that correctly
14:03 jnthn Which bit? :)
14:03 brrt the entries-into-the-handlers-extent-list-that-mark-the-boundaries-of-an-inline
14:04 jnthn Oh
14:04 jnthn It's just putting extra rows into the handlers table
14:04 jnthn So that when we walk it, we can understand that we are leaving a region of code that was inlined
14:28 brrt joined #moarvm
15:21 geospeck joined #moarvm
16:18 brrt joined #moarvm
17:18 Geth ¦ MoarVM/inline-lastexpayload: c6bee06042 | (Jonathan Worthington)++ | 2 files
17:18 Geth ¦ MoarVM/inline-lastexpayload: Remove :noinline on lastexpayload
17:18 Geth ¦ MoarVM/inline-lastexpayload: review: https://github.com/MoarVM/MoarVM/commit/c6bee06042
17:18 Geth ¦ MoarVM/inline-lastexpayload: f60ceaa07c | (Jonathan Worthington)++ | 4 files
17:18 Geth ¦ MoarVM/inline-lastexpayload: Eliminate inlined_and_not_lexical flag
17:18 Geth ¦ MoarVM/inline-lastexpayload:
17:18 Geth ¦ MoarVM/inline-lastexpayload: This isn't sufficient, and wasn't being set in the right way anyway,
17:18 Geth ¦ MoarVM/inline-lastexpayload: thus hiding another bug. It will be replaced with a more correct and
17:18 Geth ¦ MoarVM/inline-lastexpayload: more precise mechanism.
17:19 Geth ¦ MoarVM/inline-lastexpayload: review: https://github.com/MoarVM/MoarVM/commit/f60ceaa07c
17:28 Geth ¦ MoarVM/inline-lastexpayload: db85b77a26 | (Jonathan Worthington)++ | 4 files
17:28 Geth ¦ MoarVM/inline-lastexpayload: Re-work lexical handler lookup with inlines
17:28 Geth ¦ MoarVM/inline-lastexpayload:
17:28 Geth ¦ MoarVM/inline-lastexpayload: The previous mechanism was a total cheat, and only survived because
17:28 Geth ¦ MoarVM/inline-lastexpayload: marking lastexpayload as :noinline meant we never ran into any of the
17:28 Geth ¦ MoarVM/inline-lastexpayload: cases that it didn't handle. This also prevented a lot of desirable
17:28 Geth ¦ MoarVM/inline-lastexpayload: inlinings from taking place. This corrects the handling of lexical
17:28 Geth ¦ MoarVM/inline-lastexpayload: handler resolution to work with inlines.
17:28 Geth ¦ MoarVM/inline-lastexpayload: review: https://github.com/MoarVM/MoarVM/commit/db85b77a26
17:30 jnthn That took some doing.
17:34 jnthn BTW, commit message is short but the code contains some details on things
17:34 jnthn Uh, code comments
17:38 jnthn m: say 5.27 / 6.49
17:38 camelia rakudo-moar f3b1289fd: OUTPUT: «0.812018␤»
17:39 jnthn m: say 8.47 / 9.62
17:39 camelia rakudo-moar f3b1289fd: OUTPUT: «0.880457␤»
17:40 jnthn Hm, 19% and 12% performance win on a couple of array-related benchmarks because postcircumfix:<[ ]> now inlines
17:43 ilmari nice
17:45 Geth ¦ MoarVM/inline-lastexpayload: 04e291170e | (Jonathan Worthington)++ | 2 files
17:45 Geth ¦ MoarVM/inline-lastexpayload: Allow throwpayloadlex to be inlined
17:45 Geth ¦ MoarVM/inline-lastexpayload:
17:45 Geth ¦ MoarVM/inline-lastexpayload: The limitations that prevented this before are now gone.
17:45 Geth ¦ MoarVM/inline-lastexpayload: review: https://github.com/MoarVM/MoarVM/commit/04e291170e
17:45 jnthn One more win too
17:45 jnthn That one probably helps NQP code more than anything
17:45 jnthn Maybe compilation gets a tad faster
17:46 jnthn Spesh stressing seems happy
17:46 Geth ¦ MoarVM/master: 4 commits pushed by (Jonathan Worthington)++
17:46 Geth ¦ MoarVM/master: c6bee06042 | Remove :noinline on lastexpayload
17:46 Geth ¦ MoarVM/master: f60ceaa07c | Eliminate inlined_and_not_lexical flag
17:46 Geth ¦ MoarVM/master: db85b77a26 | Re-work lexical handler lookup with inlines
17:46 Geth ¦ MoarVM/master: 04e291170e | Allow throwpayloadlex to be inlined
17:46 Geth ¦ MoarVM/master: review: https://github.com/MoarVM/MoarVM/compare/0406d43143...04e291170e
17:46 jnthn So, there's the merge
17:49 * jnthn wanders home
17:54 timotimo fantastic!
17:55 * timotimo sees a ~27 line comment
17:56 Geth ¦ MoarVM: fac29990d1 | (Timo Paulssen)++ | src/profiler/heapsnapshot.c
17:56 Geth ¦ MoarVM: revert accidental half-inclusion of incremental heap snapshot data
17:56 Geth ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/commit/fac29990d1
18:11 domidumont joined #moarvm
18:51 Ven joined #moarvm
19:18 benchable6 joined #moarvm
19:35 greppable6 joined #moarvm
19:35 releasable6 joined #moarvm
19:35 statisfiable6 joined #moarvm
19:35 reportable6 joined #moarvm
19:36 unicodable6 joined #moarvm
19:36 bisectable6 joined #moarvm
19:36 committable6 joined #moarvm
19:55 geospeck joined #moarvm
20:21 Ven`` joined #moarvm
20:52 geospeck joined #moarvm
21:18 Ven`` joined #moarvm
21:26 Ven`` joined #moarvm
21:45 travis-ci joined #moarvm
21:45 travis-ci MoarVM build failed. Jonathan Worthington 'Allow throwpayloadlex to be inlined
21:45 travis-ci https://travis-ci.org/MoarVM/MoarVM/builds/309632662 https://github.com/MoarVM/MoarVM/compare/0406d431437b...04e291170ec6
21:45 travis-ci left #moarvm
22:54 jnthn Tripped over the version bump
23:12 timotimo hm, we should be able to recognize that kind of failure with our bot, except the bot isn't in here
23:12 timotimo buggable, i think it is
23:12 timotimo wait, buggable *is* here?
23:47 Ven`` joined #moarvm
23:55 reportable6 joined #moarvm
23:56 reportable6 joined #moarvm

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