[00:17] <Geth> ¦ rakudo: ugexe++ created pull request #6700: RakuAST: compile a recursive sub call with a static callee lookup

[00:17] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6700

[00:57] <Geth> ¦ rakudo/main: be8107f365 | (Nick Logan)++ (committed using GitHub Web editor) | 2 files

[00:57] <Geth> ¦ rakudo/main: RakuAST: compile a recursive sub call with a static callee lookup (#6700)

[00:57] <Geth> ¦ rakudo/main: 

[00:57] <Geth> ¦ rakudo/main: Previously a call to a sub from inside its own body kept the plain

[00:57] <Geth> ¦ rakudo/main: callee lookup, while every other call to the same sub compiled to a

[00:57] <Geth> ¦ rakudo/main: static one. A routine's own scope finds the routine under its own name,

[00:57] <Geth> ¦ rakudo/main: so the bound once check saw the name declared in a scope nested inside

[00:57] <Geth> ¦ rakudo/main: the outermost one and declined. The routine is never declared in its

[00:57] <Geth> ¦ rakudo/main: <…commit message has 10 more lines…>

[00:57] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/be8107f365

[01:44] <Geth> ¦ rakudo: ugexe++ created pull request #6701: RakuAST: lower sigilless term declarations to locals

[01:44] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6701

[02:15] <[Coke]> nearly done with changelog.

[02:15] <[Coke]> releasable6: info

[02:15] <releasable6> [Coke], Next release in ≈6 days and ≈16 hours. There are no known blockers. 442 out of 468 commits logged

[02:15] <releasable6> [Coke], Details: https://gist.github.com/431fc57602feb82c55be09a3ff5b507d

[02:16] <[Coke]> anyone wants to get those last few, have fun. :)

[02:21] <ugexe> we'll want to do a rakuast blin run tomorrow

[02:23] <ugexe> none of the stuff from the last run will have been fixed yet, but the PR from earlier this morning was pretty big, and we might as well wait for me to get these last couple of optimizations out tonight before running it again

[02:27] <ugexe> we also probably want to get https://github.com/MoarVM/MoarVM/pull/2044 in

[02:33] <Geth> ¦ rakudo/main: a2465acccf | (Nick Logan)++ | 5 files

[02:33] <Geth> ¦ rakudo/main: RakuAST: lower sigilless term declarations to locals

[02:33] <Geth> ¦ rakudo/main: 

[02:33] <Geth> ¦ rakudo/main: Previously the lexical to local lowering only considered sigiled

[02:33] <Geth> ¦ rakudo/main: declarations, term parameters and self, so `my \x := ...` always stayed

[02:33] <Geth> ¦ rakudo/main: a by-name lexical. The setting declares most of its temporaries this

[02:33] <Geth> ¦ rakudo/main: way, which left routines like DIVIDE_NUMBERS doing a lexical bind and

[02:33] <Geth> ¦ rakudo/main: several lexical reads per call where the legacy optimizer used

[02:33] <Geth> ¦ rakudo/main: <…commit message has 8 more lines…>

[02:33] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/a2465acccf

[02:33] <Geth> ¦ rakudo/main: febf0e76c4 | (Nick Logan)++ | 2 files

[02:33] <Geth> ¦ rakudo/main: Keep the Rat overflow lookup out of CREATE_RATIONAL_FROM_INTS

[02:33] <Geth> ¦ rakudo/main: 

[02:33] <Geth> ¦ rakudo/main: Previously the candidate that builds a Rat read $*RAT-OVERFLOW in its

[02:33] <Geth> ¦ rakudo/main: overflow arm. RakuAST compiles a dynamic variable to an inline lookup

[02:33] <Geth> ¦ rakudo/main: op, and MoarVM does not inline a frame that contains one, so every Rat

[02:33] <Geth> ¦ rakudo/main: operation paid a real call to create its result even though the

[02:33] <Geth> ¦ rakudo/main: overflow arm almost never runs.

[02:33] <Geth> ¦ rakudo/main: 

[02:33] <Geth> ¦ rakudo/main: This moves the lookup to a Rakudo::Internals method. The candidate is

[02:34] <Geth> ¦ rakudo/main: inlined into DIVIDE_NUMBERS again when CORE is compiled by RakuAST, and

[02:34] <Geth> ¦ rakudo/main: nothing changes for the legacy frontend, which calls DYNAMIC instead.

[02:34] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/febf0e76c4

[02:34] <Geth> ¦ rakudo/main: 153cfad5aa | (Nick Logan)++ (committed using GitHub Web editor) | 7 files

[02:34] <Geth> ¦ rakudo/main: Merge pull request #6701 from ugexe/ugexe/rakuast-lower-term-declarations

[02:34] <Geth> ¦ rakudo/main: 

[02:34] <Geth> ¦ rakudo/main: RakuAST: lower sigilless term declarations to locals

[02:34] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/153cfad5aa

[03:00] <releasable6> Next release in ≈6 days and ≈15 hours. There are no known blockers. Please log your changes in the ChangeLog: https://github.com/rakudo/rakudo/wiki/ChangeLog-Draft

[03:15] <Geth> ¦ rakudo: ugexe++ created pull request #6702: RakuAST: don't allocate a container for a lowered parameter

[03:15] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6702

[04:09] <Geth> ¦ rakudo/main: 4 commits pushed by (Nick Logan)++

[04:09] <Geth> ¦ rakudo/main: be5e720d6d | RakuAST: declare a lowered parameter's local without a container

[04:09] <Geth> ¦ rakudo/main: 90276fe4d1 | RakuAST: lower WhateverCode parameters to locals

[04:09] <Geth> ¦ rakudo/main: 5c6c7b4cc7 | Skip start-slip when a map block returns Empty

[04:09] <Geth> ¦ rakudo/main: ec79f3786a | Merge pull request #6702 from ugexe/ugexe/rakuast-parameter-local-no-container

[04:09] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/153cfad5aa0c...ec79f3786a83

[04:33] <Geth> ¦ rakudo: ugexe++ created pull request #6703: Assign to a new Array or Hash element from its own callsite

[04:33] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6703

[09:02] *** apogee_ntv left
[09:08] *** finanalyst joined
[10:18] *** apogee_ntv joined
[10:18] *** shareable6 left
[10:19] *** shareable6 joined