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