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
|