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:02 reportable6 left 00:05 reportable6 joined 01:05 greppable6 left, reportable6 left, unicodable6 left, bloatable6 left, coverable6 left, benchable6 left, notable6 left, releasable6 left, sourceable6 left, nativecallable6 left, statisfiable6 left, committable6 left, evalable6 left, tellable6 left, linkable6 left, bisectable6 left, quotable6 left, shareable6 left 01:06 statisfiable6 joined 01:07 linkable6 joined, shareable6 joined, coverable6 joined, bloatable6 joined 01:08 releasable6 joined 01:12 squashable6 joined 02:06 committable6 joined, evalable6 joined, benchable6 joined 02:07 reportable6 joined, nativecallable6 joined 02:08 tellable6 joined, sourceable6 joined 03:06 notable6 joined 03:07 unicodable6 joined, bisectable6 joined 03:24 squashable6 left 03:26 squashable6 joined 03:34 squashable6 left 03:35 squashable6 joined 04:07 greppable6 joined 04:08 quotable6 joined 06:02 reportable6 left, reportable6 joined 07:10 Altai-man joined 07:12 Altai-man left 07:14 Altai-man joined, sena_kun left 09:09 evalable6 left, linkable6 left 09:10 linkable6 joined 09:12 evalable6 joined 10:12 linkable6 left 10:13 linkable6 joined 10:42 Altai-man left 11:42 frost joined 12:02 reportable6 left 13:04 reportable6 joined 14:08 frost left
nine dogbert17: I wonder how you came up with that commit and diff? Neither reverting that part, nor reverting the whole commit fixes things for me. In fact I have to revert almost everything including "Allow inlining of dispatches with resume init args" to get it working. 15:22
tellable6 2021-09-12T20:26:30Z #raku-dev <vrurg> nine Could you, please, merge github.com/niner/Inline-Perl5/pull/168 because R#4495 depends on it.
15:23 linkable6 left 15:24 linkable6 joined 15:27 sena_kun joined
dogbert17 nine: welcome back 15:41
dogbert17 tries to figure out what nine is referring to ...
if it's the callframe diff you're referring to then it seems that some comments, from me, should be taken with a grain of salt :( 15:43
nine Ah, yes, talking about callframe 15:44
dogbert17 the diff should be ignored but if I didn't mess up completely going back to commit github.com/MoarVM/MoarVM/commit/e3...c059a8ce88 seems to fix the problem 15:45
dogbert17 famous last words
nine Sorry, can't confirm 15:46
dogbert17 let's assume that I was speaking nonsense then :)
dogbert17 retries 15:47
hah, I messed up again my comment about 3c6d39dcd4 fixing the problem is incorrect, that's the commit which seems to introduce the problem 16:08
on the previous commit , i.e. a4853302b9ea8 I'm unable to repro the callfram problem. Perhaps I'm not trying hard enough. 16:09
s/3c6d39dcd4/e3c6d39dcd4/ 16:10
nine: did you have a nice vacation? 16:12
16:38 patrickb joined
nine dogbert17: oh yes, very much so, thanks :) 17:11
dogbert17: oh, indeed! a4853302b9ea83a63e941a935055b9a9225ce233 does look fine 17:18
The major difference is that in the broken version EXISTS-KEY gets inlined. 17:19
dogbert17 nine: sounds like you're already zeroing in on the bug 17:36
I have, what I believe to be, a much simpler bug which might perhaps be of interest at a later time 18:02
18:03 reportable6 left 18:05 reportable6 joined 18:08 patrickb left
nine dogbert17: I maz have missed that a4853302b9 is still working because from e3c6d39dc on it segfaults until a later commit 18:24
Huh...the broken version deopts in 'a' while the working version never does 18:30
That deopt happens from the sp_guardconc right before the return_o (idx 20 -> pc 240) 18:50
20:20 evalable6 left, linkable6 left 20:22 evalable6 joined 20:26 nine left, nine joined 21:55 evalable6 left 22:45 vrurg joined 22:57 evalable6 joined 23:20 linkable6 joined