Welcome to the main channel on the development of MoarVM, a virtual machine for NQP and Rakudo (moarvm.org). This channel is being logged for historical purposes.
Set by lizmat on 24 May 2021.
00:08 reportable6 left 01:14 linkable6 left 03:15 linkable6 joined 04:01 AlexDaniel left, psydroid left 04:08 reportable6 joined 04:36 psydroid joined 04:58 psydroid left 05:26 rypervenche left 06:07 reportable6 left 06:10 reportable6 joined 07:56 squashable6 left, linkable6 left, reportable6 left, bloatable6 left, coverable6 left, unicodable6 left, statisfiable6 left, notable6 left, tellable6 left, releasable6 left, quotable6 left, committable6 left, nativecallable6 left, bisectable6 left, benchable6 left, shareable6 left, greppable6 left, sourceable6 left, evalable6 left 07:57 tellable6 joined, bisectable6 joined 07:58 evalable6 joined, statisfiable6 joined, coverable6 joined, reportable6 joined 07:59 squashable6 joined, quotable6 joined 08:57 nativecallable6 joined, committable6 joined, releasable6 joined 08:58 benchable6 joined, bloatable6 joined, greppable6 joined 08:59 linkable6 joined 09:57 unicodable6 joined, notable6 joined 09:58 shareable6 joined
Geth MoarVM: ecfc0bc220 | (Nicholas Clark)++ | src/core/coerce.c
Avoid reading beyond the end of the string in MVM_coerce_s_i

The recent refactoring to use `MVMCodepointIter` resulted in certain strings (that are not valid numbers) causing the code to attempting to read beyond the end of the string. For example, `nqp::coerce_si("-")` would throw "Ā«Iteration past end of grapheme iterator" instead of returning 0.
... (12 more lines)
MoarVM: 596be5246f | (Nicholas Clark)++ | src/core/coerce.c
Replace locale-"aware" <ctype.h> with ASCII-only code in MVM_coerce_s_i

We don't want (or need) implementations of isspace(c) or isdigit(c) with the overhead of locale handling.
11:27 frost joined 11:59 sourceable6 joined 12:09 reportable6 left 12:10 reportable6 joined 12:19 frost left 13:19 benchable6 left, greppable6 left, coverable6 left, unicodable6 left, shareable6 left, committable6 left, releasable6 left, sourceable6 left, quotable6 left, tellable6 left, reportable6 left, evalable6 left, linkable6 left, notable6 left, statisfiable6 left, squashable6 left, nativecallable6 left, bisectable6 left, bloatable6 left, evalable6 joined 13:20 releasable6 joined, tellable6 joined, bloatable6 joined, unicodable6 joined, benchable6 joined, quotable6 joined, greppable6 joined 13:21 sourceable6 joined, statisfiable6 joined 13:22 bisectable6 joined, linkable6 joined
MasterDuke nine: did you happen to notice logs.liz.nl/raku/2022-01-29.html#17:38-0002 ? 13:57
nine No, missed that due to server restart 14:00
14:20 committable6 joined 14:21 squashable6 joined
nine nqp: my $a := MAST::Bytecode.new; $a.write_uint32_at((my uint $b := 4), 0); # this is how it's meant to be 14:31
camelia ( no output )
15:20 reportable6 joined, notable6 joined 15:21 shareable6 joined 15:22 nativecallable6 joined 15:50 Geth left 15:51 Geth joined, RakuIRCLogger__ left
MasterDuke ah, right. LTA error though 16:00
nine Well it's NQP. It's supposed to be hard :D 16:09
I know what's wrong with github.com/MoarVM/MoarVM/issues/1656 and got a fix for the segfault (but not the bug that triggers it) 16:10
Geth MoarVM/fix_segfault_after_capture_replace_literal_arg: 2abb469945 | (Stefan Seifert)++ | src/6model/reprs/MVMCapture.c
Fix segfault after capture-replace-literal-arg with different kind

A comment above MVM_capture_replace_arg already said: "The callsite argument type is expected to be the same". Sadly comments are easy to miss and when arguments are replaced with values of different kinds, we can end up with e.g. a literal number for a callsite argument with kind obj and may try to dereference what we wrongly believe to be a pointer.
Fix by actually enforcing this rule. Fixes the segfault reported in GH #1656
MoarVM: niner++ created pull request #1658:
Fix segfault after capture-replace-literal-arg with different kind
MasterDuke `Frame has invalid bytecode start point 207994 (size 40358)` never seen this error before (this isn't on master, i'm playing around with pre-sizing the byte array we write frames into) 16:26
dogbert17 I see that nine is back in action. Welcome back :) 16:46
nine Thanks :) Nice knowing that I've not completely rusted in in the snow 16:49
Nicholas you have snow? All we have is wind. 16:50
I am really glad that you have located the cause of that bug as it was completely beyond me. 16:51
jnthnwrthngtn A lot of wind here too. The plant pots are all still on the terrace.
Nicholas aha :-)
yes. One of ours has made a leap from a low wall. A couple of times.
(over the past month) 16:52
nine Well they did have a lot of snow where we were vacationing. Now back to the usual brownish green
Nicholas aha right.
dogbert17 nine: github.com/MoarVM/MoarVM/issues/1628 has indeed been fixed 17:03
17:22 coverable6 joined
MasterDuke ugh, had to miss my usual week snowboarding these past two years. did do a long weekend in early 2020. we were talking about doing a long weekend next month, but it's tough with small kids anyway and then the crazy numbers in europe just put a nail in it 17:27
but i'm hoping i get out to the usual week next year
nine dogbert17: excellent 17:48
18:07 reportable6 left 18:51 AlexDaniel joined 19:03 psydroid joined 19:09 reportable6 joined 19:17 psydroid left 19:19 AlexDaniel left 20:27 linkable6 left, evalable6 left, evalable6 joined
MasterDuke anyone mind taking a look at this diff and trying to spot what i'm missing? gist.github.com/MasterDuke17/9114b...5e3da1fc0d 20:46
for some reason $indx isn't ending up with the correct value
nine can't see anything obvious 21:55
MasterDuke what i find odd is that when it's wrong, the guess (new size) is high. it should only ever be low (since i have to adjust for the branch in the `@handlers` case manually) 22:01
Uploaded file: uploads.kiwiirc.com/files/eb362ecb...pasted.txt 23:33
timo: you might find blog.janestreet.com/magic-trace/ interesting 23:34