[03:20] *** bisectable6 left [03:20] *** nativecallable6 left [03:20] *** shareable6 left [03:20] *** coverable6 left [03:20] *** greppable6 left [03:20] *** tellable6 left [03:20] *** benchable6 left [03:20] *** unicodable6 left [03:20] *** squashable6 left [03:20] *** evalable6 left [03:20] *** sourceable6 left [03:21] *** benchable6 joined [03:21] *** bisectable6 joined [03:22] *** squashable6 joined [03:22] *** sourceable6 joined [03:22] *** evalable6 joined [03:22] *** tellable6 joined [03:22] *** coverable6 joined [03:23] *** greppable6 joined [03:23] *** nativecallable6 joined [03:23] *** unicodable6 joined [03:23] *** shareable6 joined [04:24] *** benchable6 left [04:24] *** bisectable6 left [04:24] *** sourceable6 left [04:24] *** releasable6 left [04:24] *** coverable6 left [04:24] *** evalable6 left [04:24] *** shareable6 left [04:24] *** greppable6 left [04:24] *** nativecallable6 left [04:24] *** tellable6 left [04:24] *** unicodable6 left [04:24] *** squashable6 left [04:24] *** quotable6 left [04:24] *** notable6 left [04:24] *** linkable6 left [04:24] *** bloatable6 left [04:24] *** committable6 left [04:24] *** statisfiable6 left [04:24] *** squashable6 joined [04:24] *** shareable6 joined [04:25] *** tellable6 joined [04:25] *** committable6 joined [04:25] *** sourceable6 joined [04:25] *** coverable6 joined [04:25] *** greppable6 joined [04:26] *** quotable6 joined [04:26] *** bloatable6 joined [04:26] *** linkable6 joined [04:26] *** evalable6 joined [04:26] *** bisectable6 joined [04:27] *** benchable6 joined [04:27] *** nativecallable6 joined [04:27] *** statisfiable6 joined [04:27] *** unicodable6 joined [04:27] *** notable6 joined [04:27] *** releasable6 joined [07:25] *** domidumont joined [08:30] good *able6, #moarvm [08:31] *** Altai-man_ left [08:56] *** vrurg_ joined [08:58] *** vrurg left [09:07] *** sena_kun joined [09:31] *** MasterDuke joined [09:44] *** avar joined [09:51] Yeah, *able6 is what we have most of here... [09:53] who need people when we have useful bots? [10:01] shouldn't https://github.com/MoarVM/MoarVM/pull/1344/files#diff-651fd580717e78e6edebd0e8bca68377ec2f665d843c91b180dc75f45e3230cbR354 come after https://github.com/MoarVM/MoarVM/pull/1344/files#diff-651fd580717e78e6edebd0e8bca68377ec2f665d843c91b180dc75f45e3230cbR365-R366 ? it's in the same order as before converting to a REPR, but especially now [10:01] that we're going to be removing things, isn't that unsafe? [10:10] *** MasterDuke left [10:20] *** MasterDuke joined [10:35] interesting. at local HEAD (so adding in the removing of spesh candidates) now get a segv in deopt_frame, from `f->static_info->body.bytecode` where is f->static_info is 0x0 [10:39] oh [10:41] in MVM_spesh_deopt_one it's in the `if (f->spesh_cand) {` branch calling deopt_frame. but then in deopt_frame it's in the else of `if (f->spesh_cand && f->spesh_cand->body.inlines) {` and when the segv happens, not only is f->static_info 0x0, but so is f->spesh_cand [10:43] ah, well the entire *f is 0xo [10:46] but one level back in the backtrace (MVM_spesh_deopt_one) `*tc->cur_frame` (which is the f passed to deopt_frame) is not 0x0 [11:09] *** avar left [11:09] *** avar joined [11:49] MasterDuke: in what way would it be usafe? There shouldn't be anything different for adding candidates should there? [11:53] well, jnthn recommended doing roughly the same thing in MVM_spesh_candidate_discard_one. move `spesh->body.spesh_cands_and_arg_guards     = new_cands_and_arg_guards;` to after `MVM_spesh_arg_guard_regenerate(tc, &(new_cands_and_arg_guards->spesh_arg_guard), new_cands_and_arg_guards->spesh_candidates, new_num);` [11:54] which i believe fixed a segv (and left me with the invalid GC-owner panic) [11:56] though now i'm getting a segv, maybe because of fixing the missing MVMROOT (one of the recent commits to the PR)? [11:57] btw, merging that PR would make things a bit more convenient for me. i could create a new branch and PR instead of having some commits on the existing branch i'm being careful not to push. hopefully would make it easier for the rest of you to see the diff and help debug [11:58] do you have any objection to merging it now? [12:37] *** ggoebel_ left [12:37] *** ggoebel_ joined [12:43] huh. there's `MVMFrame *f = tc->cur_frame;` at the very beginning of MVM_spesh_deopt_one, but then a bit of a mix of using both `f` and `tc->cur_frame` throughout https://github.com/MoarVM/MoarVM/blob/master/src/spesh/deopt.c#L280-L309 [12:52] ha. here https://github.com/MoarVM/MoarVM/blob/master/src/spesh/deopt.c#L300 `*f ` is 0x0, but `*tc->cur_frame` is just fine [12:53] any SN9 watchers here with news ? [13:33] *** ggoebel_ left [13:35] *** MasterDuke left [13:35] *** MasterDuke joined [13:38] *** ggoebel_ joined [14:24] *** zakharyas joined [14:45] *** brrt joined [15:41] *** brrt left [16:19] yet another Rakudo Weekly News hits the Net: https://rakudoweekly.blog/2021/02/01/2021-05-gsoc-proposing/ [16:19] looks like the earliest SN9 launch will be on Tue [16:21] MasterDuke: merge is ok with me [16:26] nine: cool, thanks [16:27] *** brrt joined [16:29] fwiw, new-disp isn't just about multiple dispatch, it's about All The Dispatch :) [16:30] And even things we didn't actually use to consider to be dispatch [16:30] so how would you phrase it ? [16:32] *** brrt` joined [16:32] *** brrt left [16:32] All dispatch | Such unified | Better [16:33] there, someone on the Internet is wrong. This will need correcting :-) [16:35] changed it to "all dispatching" [16:36] I don't see nine in the fosdem schedule [16:41] "all dispatching" is fine [16:41] I plan to make a blog post when I get a bit further along [16:56] that's probably nine is too busy atm to do presentations [16:56] hm. if i "squash and merge" through the github ui, is it going to pull in all the commit messages? or do i manually need to that (or write a comprehensive one in the "extended description" section)? [16:58] * nine would never trust a web UI with that... [16:59] I'm pertty sure it doesn't take all the commit messages [17:00] oh! then maybe i'd better manually squash them and massage the commit message(s) and then just do a regular merge [17:01] To me the retention of commit messages is usually a bit more important than "every commit builds", in that git bissect --first-parent means that we can avoid looking inside of branches and bissect only their merges [17:02] huh, no idea it worked that way [17:03] hm, wonder if that would be a good thing to implement for bisectable6 [17:05] *** brrt` is now known as brrt [17:06] like i mentioned in a comment on the PR, i wish there was a way to attach messages to individual parts of a larger diff that don't really make sense as a comment. i.e., they're about the change in the code, not the final state [17:28] *** domidumont left [17:33] *** brrt left [17:44] *** [Coke] joined [17:44] *** MasterDuke left [17:49] *** domidumont joined [18:05] *** domidumont left [19:23] *** vrurg_ is now known as vrurg [19:24] *** vrurg left [19:25] *** zakharyas left [19:31] *** MasterDuke joined [19:38] *** MasterDuke left [19:41] *** MasterDuke joined [19:57] *** brrt joined [19:58] oh, no geth. i just merged https://github.com/MoarVM/MoarVM/pull/1344 [20:00] *** vrurg joined [20:00] *** vrurg left [20:02] MasterDuke: feels good, doesn't it? :) [20:05] well yeah, but the real goal (remove opts) is still segfaulting... [20:14] MasterDuke++ # yay [20:22] MasterDuke: time to bump MoarVM? [20:23] lizmat: sure. the changes shouldn't be visible to nqp/rakudo (yet), but it'd be good to get them more testing [20:23] *** vrurg joined [20:23] ok, bumping [20:23] let me get the removing commits in a new PR so they can be looked at more easily [20:26] *** releasable6 left [20:26] *** notable6 left [20:26] *** statisfiable6 left [20:26] *** nativecallable6 left [20:26] *** bisectable6 left [20:26] *** linkable6 left [20:26] *** quotable6 left [20:26] *** greppable6 left [20:26] *** shareable6 left [20:26] *** squashable6 left [20:28] https://github.com/MoarVM/MoarVM/pull/1426 created [20:38] *** zakharyas joined [20:50] bumped [21:15] *** shareable6 joined [21:16] *** greppable6 joined [21:16] *** bisectable6 joined [21:16] *** squashable6 joined [21:16] *** releasable6 joined [21:16] *** nativecallable6 joined [21:17] *** linkable6 joined [21:17] *** quotable6 joined [21:17] *** notable6 joined [21:17] *** statisfiable6 joined [21:26] *** zakharyas left [21:48] *** MasterDuke left [23:39] *** brrt left