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.
timo !cursor_start uses the callercode op, which is currently :noinline, can't we relatively easily transform this when inlining in spesh though? 10:27
lizmat oooh... that would be a relative hot spot for regex heavy code 10:28
timo i did the trivial implementation and it doesn't explode violently 11:19
Files=1355, Tests=117980, 83 wallclock secs (12.10 usr 2.20 sys + 1393.01 cusr 173.75 csys = 1581.06 CPU)
damn this cpu is fine
lizmat indeed :) 11:21
timo now i don't actually know in what way things would break if callercode would result in the wrong thing 11:24
gist.github.com/timo/e0710330b99df...4b80b7103c can you apply this patch to your moarvm and do some general testing, and maybe measurements with core setting compilation and other grammar-related tasks? check the inlining stats and whatnot? still have lots of dayjob stuff to do 11:40
Geth MoarVM/callercode_in_inline_is_just_curcode: e15c55036a | (Timo Paulssen)++ | 3 files
callercode no longer prevents inlining

assuming that an inliner's curcode is what an inlinee's callercode is supposed to return, we can just turn callercode into curcode when inlining. There is already a guard that prevents an inlining block from immediately inlining another so we can't for example by accident skip over one level of ... (48 more lines)
20:36
MoarVM: timo++ created pull request #1753:
callercode no longer prevents inlining
timo lizmat: have you had time to do some testing?
lizmat timo: no 22:11
timo OK! that's no problem :) 22:12
if nothing asplodes in the spec tests, i'm sure it's fine :P :P :P :P
but no really i would like to have someone who's had to deal with inlining issues and wrote spesh code before review this, maybe nine has a moment to poke at it 22:13
moarvm.org's "features" page and the readme have outdated unicode versions, they say 13.1 but in january we got support for 15 afaict 22:15
Geth MoarVM/main: 80559738e1 | (Timo Paulssen)++ | README.markdown
mention we have unicode 15
22:17
lizmat sleep& 22:31