00:00 bisectable6 joined, committable6 joined 00:29 evalable6 joined 00:30 bisectable6 joined, committable6 joined 00:59 evalable6 joined 01:00 bisectable6 joined, committable6 joined 01:13 committable6 joined 01:43 xi- joined 02:03 xi- joined 02:18 zakharyas joined 02:59 ilbot3 joined 06:39 brrt joined
Geth MoarVM: briandfoy++ created pull request #759:
Update Unicode version
06:42
brrt good * #moarvm 06:46
06:53 domidumont joined 06:59 domidumont joined
Geth MoarVM: deddb916cd | (brian d foy)++ (committed by Samantha McVey) | README.markdown
Update Unicode version cited in README

The moarvm.org page touts Unicode 10, so update the README to be consistent.
07:08
MoarVM: 0406d43143 | (Samantha McVey)++ (committed using GitHub Web editor) | README.markdown
Merge pull request #759 from briandfoy/patch-1

Update Unicode version cited in README
07:08 geospeck joined
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:09
07:10 domidumont joined 07:50 brrt joined 09:18 Voldenet joined 09:33 geospeck joined 09:40 zakharyas joined, BinGOs joined, BooK joined 09:42 domidumont joined 09:57 domidumont joined 10:12 mtj_ joined
dogbert2 o/ 10:14
dogbert2 wonders if jnthn has any brane today :)
10:15 Geth joined, SourceBaby joined, synopsebot joined, dalek joined
jnthn also :) 10:38
nwc10 well, we have snow 10:39
no idea about braaaaaaaaaaaanes
jnthn Darn, the snow missed here. 10:40
dogbert2 the snow where I'm at melted away
ilmari it was down to freezing here overnight, but no snow, and it's up to 3Ā°C now 10:48
jnthn works on a Cro middleware task as a warm-up :) 10:49
Well, that went to plan :) 11:13
dogbert2 an easy fix then ? 11:17
jnthn Was actually implementing a feature, for once, rather than fixing something ;) 11:20
dogbert2 cool 11:21
brrt good *, btw 11:29
dogbert2 hello brrt 11:30
brrt i note that i, for one, do not have any brane left
they were all out this morning
dogbert2 perhaps jnthn stole your brane :)
brrt i blame the lack of snow 11:31
jnthn Grr, this inlines/handlers thing is going to be quite tricky 11:32
11:41 yoleaux joined 11:43 travis-ci joined
travis-ci MoarVM build errored. Samantha McVey 'Merge pull request #759 from briandfoy/patch-1 11:43
travis-ci.org/MoarVM/MoarVM/builds/309367478 github.com/MoarVM/MoarVM/compare/d...06d431437b
11:43 travis-ci left
jnthn Really so. Just discovering the number of places that assume a flat table. 11:46
And noting that each of them will get more complex. 11:49
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 11:52
12:02 yoleaux joined
jnthn lunch & 12:08
12:29 AlexDaniel` joined 12:50 zakharyas joined 12:54 harrow joined 13:01 ilmari[m] joined 13:02 lizmat joined 13:15 lizmat joined
jnthn om nom borsch 13:16
So I think I have a far easier to implement alternative solution
nwc10 \o/
jnthn We poke entries into the handlers extent list that mark the boundaries of an inline
Thus the handler walker knows when it's crossing such a boundary 13:17
That provides a way forward on the "skip the caller, then look for outer" issue 13:18
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:19
src/strings/unicode.c: In function ā€˜MVM_unicode_get_property_strā€™: 13:24
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;
samcv: ^
brrt jnthn, i'm not sure if i understood that correctly 13:48
jnthn Which bit? :) 14:03
brrt the entries-into-the-handlers-extent-list-that-mark-the-boundaries-of-an-inline
jnthn Oh 14:04
It's just putting extra rows into the handlers table
So that when we walk it, we can understand that we are leaving a region of code that was inlined
14:28 brrt joined 15:21 geospeck joined 16:18 brrt joined
Geth MoarVM/inline-lastexpayload: c6bee06042 | (Jonathan Worthington)++ | 2 files
Remove :noinline on lastexpayload
17:18
MoarVM/inline-lastexpayload: f60ceaa07c | (Jonathan Worthington)++ | 4 files
Eliminate inlined_and_not_lexical flag

This isn't sufficient, and wasn't being set in the right way anyway, thus hiding another bug. It will be replaced with a more correct and more precise mechanism.
MoarVM/inline-lastexpayload: db85b77a26 | (Jonathan Worthington)++ | 4 files
Re-work lexical handler lookup with inlines

The previous mechanism was a total cheat, and only survived because marking lastexpayload as :noinline meant we never ran into any of the cases that it didn't handle. This also prevented a lot of desirable inlinings from taking place. This corrects the handling of lexical handler resolution to work with inlines.
17:28
jnthn That took some doing. 17:30
BTW, commit message is short but the code contains some details on things 17:34
Uh, code comments
m: say 5.27 / 6.49 17:38
camelia 0.812018
jnthn m: say 8.47 / 9.62 17:39
camelia 0.880457
jnthn Hm, 19% and 12% performance win on a couple of array-related benchmarks because postcircumfix:<[ ]> now inlines 17:40
ilmari nice 17:43
Geth MoarVM/inline-lastexpayload: 04e291170e | (Jonathan Worthington)++ | 2 files
Allow throwpayloadlex to be inlined

The limitations that prevented this before are now gone.
17:45
jnthn One more win too
That one probably helps NQP code more than anything
Maybe compilation gets a tad faster
Spesh stressing seems happy 17:46
Geth MoarVM/master: 4 commits pushed by (Jonathan Worthington)++
jnthn So, there's the merge
jnthn wanders home 17:49
timotimo fantastic! 17:54
timotimo sees a ~27 line comment 17:55
Geth MoarVM: fac29990d1 | (Timo Paulssen)++ | src/profiler/heapsnapshot.c
revert accidental half-inclusion of incremental heap snapshot data
17:56
18:11 domidumont joined 18:51 Ven joined 19:18 benchable6 joined 19:35 greppable6 joined, releasable6 joined, statisfiable6 joined, reportable6 joined 19:36 unicodable6 joined, bisectable6 joined, committable6 joined 19:55 geospeck joined 20:21 Ven`` joined 20:52 geospeck joined 21:18 Ven`` joined 21:26 Ven`` joined 21:45 travis-ci joined
travis-ci MoarVM build failed. Jonathan Worthington 'Allow throwpayloadlex to be inlined 21:45
travis-ci.org/MoarVM/MoarVM/builds/309632662 github.com/MoarVM/MoarVM/compare/0...e291170ec6
21:45 travis-ci left
jnthn Tripped over the version bump 22:54
timotimo hm, we should be able to recognize that kind of failure with our bot, except the bot isn't in here 23:12
buggable, i think it is
wait, buggable *is* here?
23:47 Ven`` joined 23:55 reportable6 joined 23:56 reportable6 joined