github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
00:08 ZzZombo_ joined, p6bannerbot sets mode: +v ZzZombo_ 00:10 ZzZombo left, ZzZombo_ is now known as ZzZombo 00:15 lizmat left
timotimo stagestats uses MVM_bigint_from_num 00:40
most probably because literals in nqp are nums, and if you want to create a bigint, you can't just box_I, you have to fromnum_I
that seems to be all the invocations for MVM_bigint_from_num in stage compilation 00:41
MasterDuke ugh, i'd like to make progress on my default-nqp-to-int branch, but it's not easy 00:49
timotimo yeah, i totally agree that it's hard 00:50
MasterDuke changing topics, what do you think about my suggestion to make the parameter to --profile be the filename instead of the kind, and then change --profile-filename to --profile-kind? 00:54
timotimo actually i started - long ago - on a branch that lets you put more parameters in the --profile argument 01:07
i wanted that for the heap snapshot profiler so that it could be told to only take snapshots when there is a major collection
since sometimes there are hundreds of minor collections in a row, and that leads to extremely big files 01:08
would you say that should go into --profile-kind as well, then? 01:09
i.e. switch them completely?
01:16 fake_space_whale joined
MasterDuke yeah, i'd say switch them 01:16
01:17 p6bannerbot sets mode: +v fake_space_whale
MasterDuke at least personally, i've created maybe 3 heap profiles ever 01:17
and maybe 300 profiles that i had to use --profile-filename for
not very huffmanized for my use 01:18
01:20 avar left, avar joined, avar left, avar joined, p6bannerbot sets mode: +v avar 01:21 p6bannerbot sets mode: +v avar
timotimo that does make sense 01:22
it'd be good to have a transition period where both styles are allowed
MasterDuke you think there are that many people creating heap profiles? 01:25
timotimo hm 01:30
01:40 NikitaKonovalov2 joined 01:41 p6bannerbot sets mode: +v NikitaKonovalov2 01:43 NikitaKonovalov2 left 02:01 mystica55512 joined, p6bannerbot sets mode: +v mystica55512 02:02 mystica55512 left 02:06 optiz0r0 joined, p6bannerbot sets mode: +v optiz0r0 02:07 optiz0r0 left 02:19 ZzZombo left 02:37 \13k29 joined 02:38 p6bannerbot sets mode: +v \13k29, \13k29 left
timotimo with an example like .words() where the default is Inf, spesh might be able to turn the "while $count < $limit" into essentially "loop" 02:41
no, i mean "repeat"
timotimo goes to bed 02:42
02:48 Guest93338 joined 02:49 p6bannerbot sets mode: +v Guest93338, Guest93338 left 03:06 ZzZombo joined, p6bannerbot sets mode: +v ZzZombo 03:16 johnlage11 joined, p6bannerbot sets mode: +v johnlage11 03:21 johnlage11 left 03:34 smooth1 joined 03:35 p6bannerbot sets mode: +v smooth1 03:36 smooth1 left 03:43 Bink27 joined, p6bannerbot sets mode: +v Bink27 03:45 Bink27 left 05:42 fake_space_whale left 05:45 SirFunk_ joined 05:46 p6bannerbot sets mode: +v SirFunk_ 05:50 SirFunk_ left 06:05 domidumont joined 06:06 p6bannerbot sets mode: +v domidumont 06:07 domidumont1 joined 06:08 p6bannerbot sets mode: +v domidumont1 06:10 domidumont left 06:17 domidumont1 left 06:23 chrisdickinson13 joined, fake_space_whale joined 06:24 p6bannerbot sets mode: +v chrisdickinson13, p6bannerbot sets mode: +v fake_space_whale 06:26 chrisdickinson13 left 06:29 patrickb joined, p6bannerbot sets mode: +v patrickb 06:34 patrickb left 06:43 robertle joined 06:44 p6bannerbot sets mode: +v robertle 06:48 fake_space_whale left 07:00 lizmat joined, p6bannerbot sets mode: +v lizmat 07:16 lizmat left, domidumont joined 07:17 p6bannerbot sets mode: +v domidumont 07:38 patrickb joined, p6bannerbot sets mode: +v patrickb 07:48 ZzZombo_ joined, p6bannerbot sets mode: +v ZzZombo_ 07:50 ZzZombo left, ZzZombo_ is now known as ZzZombo 07:59 robertle left 08:01 robertle joined 08:02 p6bannerbot sets mode: +v robertle 08:08 m3point1428 joined, m3point1428 left 08:24 lizmat joined, p6bannerbot sets mode: +v lizmat 08:29 robertle left, robertle joined 08:30 p6bannerbot sets mode: +v robertle 08:35 robertle left, Milo-17 joined 08:36 p6bannerbot sets mode: +v Milo-17 08:37 robertle joined, brrt joined 08:38 p6bannerbot sets mode: +v robertle, p6bannerbot sets mode: +v brrt 08:39 Milo-17 left
brrt \o 08:42
timotimo: I have a *very* similar branch I was working on :-) 08:43
08:47 Sigyn joined, Sigyn left 08:50 Sigyn joined, p6bannerbot sets mode: +v Sigyn 08:51 Sigyn left 08:56 Sigyn joined, p6bannerbot sets mode: +v Sigyn
lizmat Q: what *is* the difference between nqp::assign and nqp::p6assign? 08:59
brrt (no idea)
lizmat apparently, according to commit message, the p6 version is using a spesh plugin 09:00
we have 4 cases of nqp::assign in core: I wonder if those should be changed as well ? 09:01
timotimo don't we also have something better than assign + scalarfromdesc? 09:04
09:06 blackbeard4201 joined
lizmat I believe so: that piece of code hasn't been touched in 2 years 09:06
jnthn nqp::p6assign uses the spesh plugin (on MoarVM, equivalent to assign otherwise)
brrt is about to remove the jit log file altogether
09:06 p6bannerbot sets mode: +v blackbeard4201, ozfalcon joined
jnthn I think all cases can be updated, *but* as timotimo mentioned some cases have better options than a direct replacement 09:07
09:07 blackbeard4201 left
lizmat jnthn: so it is safe to just s/assign/p6assign/ ? 09:07
09:07 p6bannerbot sets mode: +v ozfalcon
jnthn Safe but it's worth looking at each one 09:07
lizmat or maybe just rewrite it using = ?
jnthn Because there's a scalarwithvalue op that combines creation and initialization and skips the whole handling of default value, because we know it'll never be read 09:08
Which is a nice saving
lizmat and if the descriptor is nqp::null ? 09:09
jnthn Then assign will always fail?
lizmat "nqp::assign(nqp::p6scalarfromdesc(nqp::null),iterable)"
been sitting in the code for 2 years without any spectest fails :-( 09:10
jnthn Oh, right, that depends on using the default descriptor...I think I found a better way of handling that in most cases.
Since it means a null check and conditional every time
09:11 ozfalcon left
jnthn Anyway, the reason I didn't just blanket change them is so they could be each carefully audited for the optimal change. 09:11
lizmat right
lemme see if I can do such an audit 09:12
jnthn See in Actions.nqp for the various desugars available
09:13 marxjohnson joined, marxjohnson left
lizmat I was looking at Array::ASSIGN-POS 09:13
jnthn I'm very sure that's one I audited... 09:14
Yes, looks like that's been carefully crafted indeed :) 09:15
To be just under the inline size limit :) 09:16
Geth MoarVM/jit-comment-on-spesh-log: 11 commits pushed by (Timo Paulssen)++, (Bart Wiegmans)++
review: github.com/MoarVM/MoarVM/compare/2...815677f827
09:41
timotimo btw, in theory we could collect spesh logs for spec test runs and see if any frames switch between "can be inlined" and "cannot be inlined" and automatically report that 09:42
brrt aye. 09:45
timotimo: I took your branch a bit further, there's no more jit log now; I wonder if we can get away with getting rid of the bytecode directory entirely 09:46
timotimo how do you reckon? 09:53
brrt well, I mean, i don't want to have to specify it
timotimo so, always have it if the spesh log is turned on? 09:54
brrt I want something like MVM_JIT_DUMP_BYTECODE=1 and have it dump the bytecode to /tmp
no
timotimo ah
brrt well, a subdirectory of /tmp
timotimo would it go into /tmp/moarjit.PID/ or something?
brrt I can us uv_os_tmpdir
something pretty much exactly like htat
anyway, lunch & 09:55
09:55 shadowbane26 joined 09:56 p6bannerbot sets mode: +v shadowbane26 09:58 shadowbane26 left 09:59 brrt left 10:02 ZzZombo left 10:03 definity20 joined, definity20 left 10:07 zack684918 joined 10:08 p6bannerbot sets mode: +v zack684918 10:11 zack684918 left 10:13 regtools23 joined 10:14 p6bannerbot sets mode: +v regtools23
lizmat jnthn: I'm confused: should this work or not: 10:15
m: use nqp; my $a := nqp::p6bindattrinvres(nqp::create(Scalar),Scalar,q/$!value/,42); say $a; $a = 666; say $a
camelia 42
Cannot assign to a readonly variable or a value
in block <unit> at <tmp> line 1
jnthn No.
Because no descriptor means readonly
lizmat aha
10:15 regtools23 left
lizmat but, I don't have a descriptor there, so what should I put in the $!descriptor then ? 10:16
jnthn That's what I was saying early. The scalarfromdesc op has to do a null check inside of its desugar to check if the descriptor provided is null, and if so then it uses the default one instead
*earlier
lizmat so I should still be using scalarfromdesc ?
jnthn Well, you could, but perhaps better is to just obtain the default descriptor 10:17
lizmat I'm looking at [foo], there is no Array, and no descriptor yet
jnthn nqp::getcurhllsym('default_cont_spec') or some such 10:18
lizmat ack
jnthn Maybe shove a BEGIN before it so that at runtime you just have a QAST::WVal which is cheap :) 10:20
10:20 lionel22 joined
lizmat ah, I guess that can be done at some more places then 10:20
10:21 p6bannerbot sets mode: +v lionel22 10:25 lionel22 left
lizmat afk for a few hours& 10:28
dogbert2 jnthn: will there be an 'opt of the day' ? 10:30
10:36 pounard9 joined 10:37 pounard9 left 10:45 brrt joined 10:46 p6bannerbot sets mode: +v brrt 10:58 AlexDani` joined 10:59 p6bannerbot sets mode: +v AlexDani` 11:02 AlexDaniel left 11:28 ZzZombo joined, p6bannerbot sets mode: +v ZzZombo 11:30 Alforz15 joined 11:31 p6bannerbot sets mode: +v Alforz15, Alforz15 left 11:33 AlexDani` is now known as AlexDaniel 11:34 lizmat left 11:36 lifthrasiir5 joined 11:37 p6bannerbot sets mode: +v lifthrasiir5, lifthrasiir5 left 11:52 avar left, avar joined, avar left, avar joined, p6bannerbot sets mode: +v avar 11:53 p6bannerbot sets mode: +v avar 12:04 meretrix23 joined 12:05 p6bannerbot sets mode: +v meretrix23 12:07 meretrix23 left, avar left 12:08 avar joined, avar left, avar joined, p6bannerbot sets mode: +v avar 12:13 tv4 joined 12:14 p6bannerbot sets mode: +v tv4 12:17 tv4 left 12:26 copec19 joined, copec19 left 12:36 ljharb15 joined 12:37 p6bannerbot sets mode: +v ljharb15 12:41 ljharb15 left
brrt oh, good news folks 12:44
I'm looking at the libuv test suite
I think we can actually call the synchronous FS files without a loop instance
because, tests/test-fs.c does that all the time 12:45
timotimo that ought to also let us create more threads before the number of open files hits the limit
right?
brrt I'm not sure I follow
timotimo creating an uv loop costs, i believe, two file handles
brrt seriously?
timotimo i'm not 100% sure 12:46
jnthn I think timotimo is right 12:49
brrt okay, that's good news actually 12:50
I checked, and we can call all the synchronous fs stuff without a uv loop 12:51
'the tests do it so it must be OK'
12:53 butth0le joined, butth0le left 13:05 Guest1161 joined 13:06 p6bannerbot sets mode: +v Guest1161 13:09 Guest1161 left
brrt btw, samcv++ # nice post 13:21
jnthn dogbert2: There may be an opt of the day. Currently looking into the sub-optimal handling I'm seeing with spesh of array assignment... 13:27
And noticing I maybe misread... 13:28
Yeah, I did; something's maybe still wrong but not what I thought. 13:30
And ASSIGN-POS doesn't inline yet after all
dogbert2 is ASSIGN-POS too large? 13:34
jnthn Yeah
I'm wondering if I can split it
dogbert2 do you think it will be a big win if you can inline it? 13:35
13:36 Zoffix joined, p6bannerbot sets mode: +v Zoffix
Zoffix I just got "MoarVM oops: Spesh: instruction box_i missing from define-use chain" 13:36
jnthn Yes :)
Zoffix: umm...congrats... 13:37
Zoffix: Did you have the DU-chain checking mode turned on?
Zoffix I've no idea what that is, so unless it's on by default, I'm gonna say no
jnthn No, you'd have to have turned it on, but I just checked and that's the error that can be raised in normal operation. 13:38
Zoffix It's with this piece of code, when trying to run it with --profile --profile-filename=out.html gist.github.com/zoffixznet/c4fd06a...6cfd2d28db
jnthn Ah, it only happens when profiling?
Zoffix Yeah
jnthn breathes a half sigh of relief :)
Zoffix :D
jnthn I think timotimo++ already encountered this and was looking at it... 13:39
Zoffix Ah, ok
jnthn Please could you make a ticket though, so we don't lose it? 13:40
Zoffix yeah 13:41
jnthn Thanks
Zoffix M#971 13:46
synopsebot M#971 [open]: github.com/MoarVM/MoarVM/issues/971 Profiling a code gives MoarVM oops: Spesh: instruction box_i missing from define-use chain
13:46 Zoffix left 13:51 Guest56510 joined 13:52 p6bannerbot sets mode: +v Guest56510 13:55 Guest56510 left 14:02 avar left, avar joined, avar left, avar joined, p6bannerbot sets mode: +v avar 14:03 p6bannerbot sets mode: +v avar 14:09 maege- joined, p6bannerbot sets mode: +v maege- 14:10 Kaiepi left 14:11 Kaiepi joined, maege- left, p6bannerbot sets mode: +v Kaiepi
Geth MoarVM/jit-comment-on-spesh-log: fe7ea0c35f | (Bart Wiegmans)++ | 2 files
[JIT] Dump bytecode in /tmp

No need to specify the exact bytecode directory, if we can put it in a consistent place anyway.
14:14
14:27 Shaan727 joined 14:28 p6bannerbot sets mode: +v Shaan727 14:29 Shaan727 left 14:35 MerlinTHP23 joined, MerlinTHP23 left
Geth MoarVM: a5dd7652d4 | (Bart Wiegmans)++ | 7 files
[threadcontext] Remove per-thread libuv loop object

According to the libuv tests, we don't actually need a libuv loop object for the synchronous FS operations. Which means that we can use libuv for cross-platform portability, but we don't have to create and maintain a libuv loop structure, other than the one for the actual event loop.
14:37
14:40 bladdezz10 joined 14:41 p6bannerbot sets mode: +v bladdezz10, bladdezz10 left, pck7 joined 14:42 pck7 left
jnthn brrt++ 14:43
Geth MoarVM: bdw++ created pull request #972:
Remove JIT log file
brrt ok, dinner & 14:45
14:45 brrt left
Geth MoarVM: 1500090a38 | (Jonathan Worthington)++ | src/spesh/optimize.c
Optimize ifnonnull branches out

If we can determine which way they would branch, then eliminate them.
14:56
MoarVM: 597ccb09e3 | (Jonathan Worthington)++ | 5 files
Pass on type tuple when inlining unspecialized

In the case of a speculative inline where we have a stable type tuple, this means that we can use the argument types inside the specialization
  (previously we could only do that when we have all of the information
in facts).
15:04 fake_space_whale joined 15:05 p6bannerbot sets mode: +v fake_space_whale 15:06 Guest23564 joined 15:07 p6bannerbot sets mode: +v Guest23564 15:08 Guest23564 left 15:18 Slartibart10 joined 15:19 p6bannerbot sets mode: +v Slartibart10 15:20 lizmat joined, p6bannerbot sets mode: +v lizmat 15:23 Slartibart10 left 15:24 scode10 joined 15:25 domidumont left, scode10 left 15:29 patrickb left
jnthn m: say 2.051 / 955 15:35
camelia 0.0021476
jnthn m: say 2.051 / 0.955
camelia 2.147644
lizmat so what just got 2x as fast ? 15:39
yoleaux 12:11Z <El_Che> lizmat: "Elizabeth Mattijsen has been programming for a living since 1978 in various (mostly now defunct) programming languages." <-- I think the "dead languages" introduction will be associated with Perl 5 and 6 by casual readers. I would drop that part.
12:31Z <Zoffix> lizmat: would NQP benefit from the same BUILDALL gen you did in Rakudo? I imagine we create a ton of QAST objects. A plain `nqp::create(QAST::Op)` is 4.14x as fast as `QAST::Op.new`. I don't know if it could be genned when attributes are present, but it smells like there could be a nice win for compilation time (at least of Rakudo)
jnthn lizmat: the following: 15:40
for ^10_000 {
my @arr;
for ^1_000 {
@arr[$_] = 42;
}
}
lizmat cool
jnthn Alas, Perl 5 is still 1.78x as fast at it On My Machine (yes, I know, it gets to not have Int allocations, but still, this is what the typical user would compare) 15:41
So still some improvement to find yet :) 15:42
lizmat yeah, but I've heard plenty of people say that even if Perl 6 would be 2x as slow as Perl 5, they would go for it
so, in this benchmark, we're already there
jnthn
.oO( Yeah, but I want to win the benchmark :P :P )
Geth MoarVM: 3ea4a81a21 | (Jonathan Worthington)++ | src/spesh/plan.c
Fix poor inlining due to wrong plan sorting

We sorted the plan by stack depth, which is a decent approximation, but sometimes we could have a situation where a caller and callee planned for specialization would end up with the caller having a greater maximum observed stack depth than the callee.
If we detect such a case, give the callee one a boost, so that we'll do them in a more sensible order, leading to more inlining chances.
15:50
timotimo i want moar to get a --developer mode where instead of MVM_SPESH_LOG i can just put MSL in the environment ...
lizmat .tell El_Che adapted the "about me" to prevent possibly perceived association 15:52
yoleaux lizmat: I'll pass your message to El_Che.
15:53 aither19 joined, p6bannerbot sets mode: +v aither19
lizmat .ask Zoffix But then how would one set the attributes? nqp::bindattr's everywhere ? 15:55
yoleaux lizmat: I'll pass your message to Zoffix.
15:56 Zoffix joined, p6bannerbot sets mode: +v Zoffix
Zoffix lizmat: no, the enduser code would not change. 15:56
timotimo excuse me, but what was the context for this? 15:57
Zoffix lizmat: basically yesterday I had `Foo.new: :42x, :70bar` that was way faster in Rakudo than it was in nqp and jnthn said it's cause nqp's BUILDALL is not codegenned, so the natural thought is what if it were: would that benefit the compilation
timotimo: this: colabti.org/irclogger/irclogger_log...09-26#l690 15:58
and "<Zoffix> Aye, this version runs in 0.243s on HEAD: gist.github.com/zoffixznet/90c3c48...4d1a3e6c49 vs 4s for the other one"
m: say 4/.243
camelia 16.460905
15:59 aither19 left, Cthulhon3 joined
lizmat jnthn: is there a reason why you didn't do the same opt for the "int $pos" candidate ? 16:00
16:00 p6bannerbot sets mode: +v Cthulhon3
lizmat if not, I'll do that :-) 16:00
16:01 thaller joined
jnthn lizmat: um, no, just forgot :) 16:01
16:02 p6bannerbot sets mode: +v thaller
lizmat ok, will take care of it then :-) 16:02
16:02 thaller left
Zoffix $ time ./nqp -e 'class Foo { has $!x; has $!y }; my int $x; while $x++ < 1_000_000 { Foo.new: x => 42, y => 70 }' 16:02
real 0m4.522s
$ time ./perl6 -e 'class Foo { has $.x; has $.y }; my int $x; while $x++ < 1_000_000 { Foo.new: x => 42, y => 70 }'
real 0m1.056s
lizmat but after dinner&
Zoffix lizmat: ^ that's what I meant.
Perl 6 is way faster than NQP, so my assumption is that we can make NQP at least the same speed as Perl 6 16:03
to improve compilation speed
16:03 Cthulhon3 left
timotimo does the buildplan executor show up in profiles, i wonder? 16:04
or for that matter the constructors of QAST nodes
well, those are already partially hand-optimized
Zoffix didn't try 16:05
16:11 SicoWork2 joined 16:12 p6bannerbot sets mode: +v SicoWork2 16:15 Zoffix left 16:16 SicoWork2 left 16:18 robertle left 16:33 yankcrime17 joined 16:34 p6bannerbot sets mode: +v yankcrime17, kyle6129 joined 16:35 yankcrime17 left, p6bannerbot sets mode: +v kyle6129 16:39 kyle6129 left
Geth MoarVM: 51610604c1 | (Jonathan Worthington)++ | src/spesh/log.c
Correct "should we log parameters" check

Fixes a regression when the parameter logging was refactored recently. We can't use the standard way to check at this point, because we're doing it during invocation, and so haven't set everything up yet. All we really need to check is that there's a log to write into.
16:42
17:13 domidumont joined 17:14 p6bannerbot sets mode: +v domidumont 17:35 syntacticNaCl15 joined, p6bannerbot sets mode: +v syntacticNaCl15 17:36 syntacticNaCl15 left 17:53 gummipunkt8 joined 17:54 gummipunkt8 left 17:58 Kaiepi left, Kaiepi joined 17:59 p6bannerbot sets mode: +v Kaiepi 18:04 azeem12 joined 18:05 p6bannerbot sets mode: +v azeem12 18:08 azeem12 left 18:15 Nevolution217 joined, p6bannerbot sets mode: +v Nevolution217 18:16 faceman2 joined 18:17 bengl1 joined, p6bannerbot sets mode: +v faceman2 18:18 p6bannerbot sets mode: +v bengl1, Nevolution217 left, faceman2 left 18:21 bengl1 left, lizmat left 18:36 fake_space_whale left 18:45 sankarshan0 joined 18:46 p6bannerbot sets mode: +v sankarshan0 18:47 sankarshan0 left 18:49 fake_space_whale joined, p6bannerbot sets mode: +v fake_space_whale 18:59 Ven`` joined 19:00 p6bannerbot sets mode: +v Ven`` 19:07 Te[u]K28 joined 19:08 wlfshmn joined, p6bannerbot sets mode: +v Te[u]K28, p6bannerbot sets mode: +v wlfshmn 19:10 lizmat joined, p6bannerbot sets mode: +v lizmat, Te[u]K28 left 19:12 wlfshmn left 19:14 zgrepc joined 19:15 p6bannerbot sets mode: +v zgrepc 19:16 Ven`` left, zgrepc left 19:21 domidumont left 19:29 jitters_28 joined 19:30 p6bannerbot sets mode: +v jitters_28 19:31 McBadders0n joined 19:32 p6bannerbot sets mode: +v McBadders0n 19:33 jitters_28 left 19:36 en7ropy12 joined, p6bannerbot sets mode: +v en7ropy12 19:37 McBadders0n left 19:40 en7ropy12 left 20:00 pmarreck5 joined, p6bannerbot sets mode: +v pmarreck5 20:02 pmarreck5 left 20:11 fake_space_whale left 20:18 KittyKatt28 joined 20:19 KittyKatt28 left 20:36 bmlzootown18 joined, p6bannerbot sets mode: +v bmlzootown18 20:38 bmlzootown18 left 20:39 osa123 joined 20:40 p6bannerbot sets mode: +v osa123 20:45 osa123 left
timotimo my blind working hypothesis for the define-use-chain problem in the profiler: perhaps we're deleting an instruction twice and that's what blows it up 20:51
can't check immediately, though
20:51 zanshin9 joined 20:52 p6bannerbot sets mode: +v zanshin9, zanshin9 left 20:53 MasterDuke left 21:06 meineerde23 joined, p6bannerbot sets mode: +v meineerde23 21:08 jplock4 joined 21:09 Kaiepi left, p6bannerbot sets mode: +v jplock4 21:10 jplock4 left, meineerde23 left 21:18 MasterDuke joined, p6bannerbot sets mode: +v MasterDuke, MasterDuke left, MasterDuke joined, herbert.freenode.net sets mode: +v MasterDuke, p6bannerbot sets mode: +v MasterDuke 22:00 p0rt5 joined, p6bannerbot sets mode: +v p0rt5 22:02 p0rt5 left 22:09 fake_space_whale joined 22:10 p6bannerbot sets mode: +v fake_space_whale 22:14 jfmcarreira5 joined 22:15 p6bannerbot sets mode: +v jfmcarreira5 22:17 jfmcarreira5 left 22:44 niklasl3 joined 22:45 p6bannerbot sets mode: +v niklasl3
timotimo yeah, it's kind of being deleted twice 22:47
delete ins 137 0x7f21f8099b98 prof_allocated
delete ins 137 0x7f21f8099b58 box_i
[...]
delete ins 137 0x7f21f8099b58 box_i
MoarVM oops: Spesh: instruction box_i missing from define-use chain
22:49 evalable6 joined 22:50 p6bannerbot sets mode: +v evalable6 22:52 niklasl3 left
MasterDuke timotimo: easy fix? 22:52
timotimo i believe so 22:53
MasterDuke nice
22:56 jfe` joined
Geth MoarVM: 67f85dc711 | (Timo Paulssen)++ | src/spesh/optimize.c
optimize_prof_allocated shouldn't try to mess with writers.
22:56
timotimo this ought to do it.
22:56 p6bannerbot sets mode: +v jfe` 22:58 jfe` left 23:11 jtickle joined, p6bannerbot sets mode: +v jtickle 23:12 jtickle left 23:13 digdilem14 joined 23:14 p6bannerbot sets mode: +v digdilem14, typedrat11 joined, digdilem14 left 23:15 p6bannerbot sets mode: +v typedrat11 23:16 pod joined, thomasfedb4 joined, typedrat11 left, p6bannerbot sets mode: +v pod, p6bannerbot sets mode: +v thomasfedb4 23:20 pod left, thomasfedb4 left 23:30 Willd4 joined, p6bannerbot sets mode: +v Willd4 23:32 Willd4 left 23:55 knittl28 joined 23:56 p6bannerbot sets mode: +v knittl28