|
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
|
|||