github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm Set by AlexDaniel on 12 June 2018. |
|||
00:54
sena_kun left
01:09
sena_kun joined
01:35
sena_kun left
04:33
lucasb left
05:35
quotable6 left,
squashable6 left,
coverable6 left,
bisectable6 left,
sourceable6 left,
shareable6 left,
reportable6 left,
statisfiable6 left,
committable6 left,
unicodable6 left,
releasable6 left,
notable6 left,
bloatable6 left,
greppable6 left,
nativecallable6 left,
benchable6 left,
releasable6 joined,
greppable6 joined
05:36
sourceable6 joined,
squashable6 joined,
committable6 joined,
reportable6 joined
05:37
quotable6 joined,
statisfiable6 joined,
shareable6 joined,
benchable6 joined,
coverable6 joined,
nativecallable6 joined,
bloatable6 joined
05:38
unicodable6 joined,
notable6 joined,
bisectable6 joined
06:41
moritz_ is now known as moritz
07:44
domidumont joined
08:37
MasterDuke joined
08:42
zakharyas joined
09:14
sena_kun joined
09:29
sena_kun left
10:10
MasterDuke left
10:22
sivoais left,
sivoais joined
10:57
Guest38485 joined
12:41
zakharyas left
12:54
lucasb joined
13:15
domidumont left
|
|||
jnthn | Wow...compiling HEAD is very warning-y | 14:08 | |
14:13
sena_kun joined
14:14
sena_kun left
14:16
MasterDuke joined
|
|||
lizmat | I only see lots and lots of "loop not vectorized" | 14:18 | |
and some format specifies type 'long' but | |||
the argument has type 'MVMuint64' | |||
14:20
sena_kun joined
|
|||
jnthn | gist.github.com/jnthn/d2c700e9d192...a2e5a9bed3 fwiw | 14:25 | |
I don't have time to go digging at the moment. | |||
14:29
discord6 joined,
synopsebot joined
14:43
synopsebot_ joined
14:47
lizmat left
14:49
discord6 left
14:50
zakharyas joined
14:51
discord6 joined
14:56
synopsebot left,
sena_kun left
14:58
discord6 left,
discord6 joined
15:00
discord6 left
15:01
discord6 joined,
discord6 left,
discord6 joined
15:04
discord6 left
15:05
discord6 joined,
discord6 left,
discord6 joined
15:08
discord6 left
15:11
sena_kun joined
15:12
discord6 joined
15:16
discord6 left,
Geth left,
synopsebot_ left
|
|||
Kaiepi | can someone review github.com/MoarVM/MoarVM/pull/1222 ? | 15:19 | |
Guest38485 | jnthn, lizmat: a few more warning options have been enabled recently | 15:39 | |
nine: after some more debugging it turns out that the 'sc' is somehow cleared in this routine github.com/MoarVM/MoarVM/blob/mast...on.c#L2843 | 16:03 | ||
16:28
zakharyas left
16:30
Guest38485 left
16:39
discord6 joined,
discord6 left,
discord6 joined
16:55
sena_kun left
17:08
discord6 left
17:10
sena_kun joined
17:34
lizmat joined
18:01
lizmat left
18:12
lizmat joined
18:26
Kaiepi left,
Kaiepi joined
18:27
Kaiepi left
18:28
Kaiepi joined
|
|||
nine | jnthn: we compile with -Wall and -Wextra now. I've fixed almost all the warnings I got but different compiler (versions) will be more noisy still | 18:36 | |
jnthn | nine: Yeah, mine is a bit noisy. I'm all for more warnings in principle, just know it's easy to not spot new ones among the noise when there's a lot of them triggered. | 18:38 | |
nine | I know. That's why I've fixed several hundreds of them. I only left ~ 3 that I hope will be tackled by domain specialists. And of course those I simply do not get on my gcc | 18:39 | |
jnthn | nine++ | 18:44 | |
Will hopefully get chance to look at the ones I see soonish | 18:45 | ||
The fallthrough ones are very odd, it's a warning on something that looks like it's meant to prevent one... | |||
Anyways, home time... :) | |||
nine | Oh, seems like that fallthrough attribute is only supported on gcc >= 7 | 18:46 | |
Same for #pragma GCC diagnostic ignored "-Wimplicit-fallthrough" | 18:48 | ||
18:56
sena_kun left
19:11
sena_kun joined
19:31
Geth joined,
discord6 joined
19:41
lizmat left
20:05
bisectable6 left,
nativecallable6 left,
committable6 left,
releasable6 left,
rba left,
rba joined
20:10
releasable6 joined,
bisectable6 joined,
nativecallable6 joined,
committable6 joined
20:47
vrurg_ joined
20:50
vrurg left
20:56
sena_kun left
21:11
sena_kun joined
|
|||
nine | dogbert17: I think I got it! | 22:00 | |
or still not :( | 22:01 | ||
Or maybe there's just more issues with this code. | 22:03 | ||
One thing's certain: this line github.com/MoarVM/MoarVM/blob/mast...all.c#L189 must be moved 2 down to be outside the if just like the pushes. Otherwise we may end up with outdated pointers on the temp root stack | 22:04 | ||
dogbert17 | nine: cool | 22:21 | |
the only thing I have found is that the sc, which get trashed has the following description: site#sources/0F50B574622C7F9833911E194CCEAFFE83348161 (Inline::Perl5::Interpreter) | 22:22 | ||
it shows up many times before the crash occurs | 22:23 | ||
Native call expected return type with CPointer representation, but got a P6opaque (Scalar) | 22:34 | ||
in method p5_call_parent_gv | |||
on the positive side, the SEGV's seems to have vanished | 22:43 | ||
22:54
Kaiepi left
22:55
sena_kun left
22:56
Kaiepi joined,
Kaiepi left,
Kaeipi joined
23:01
Kaeipi left
23:02
Kaeipi joined
23:07
lizmat joined
23:09
sena_kun joined
23:44
lucasb left
|