[01:13] <Geth> ¦ rakudo: ugexe++ created pull request #6247: Add an optimize phase to the RakuAST frontend

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

[01:56] *** vrurg_ joined
[01:58] *** vrurg left
[04:26] <Geth> ¦ rakudo: ugexe++ created pull request #6248: RakuAST: give onlystar bodies a source location

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

[05:56] *** hurufu joined
[06:13] *** [Coke] left
[06:13] *** [Coke] joined
[08:37] <Geth> ¦ nqp/main: e37642d997 | (Elizabeth Mattijsen)++ | tools/templates/MOAR_REVISION

[08:37] <Geth> ¦ nqp/main: Bump MoarVM to get datagram leak fix, Zer0-Tolerance++

[08:37] <Geth> ¦ nqp/main: review: https://github.com/Raku/nqp/commit/e37642d997

[09:15] <Geth> ¦ rakudo/main: 931393bdb9 | (Elizabeth Mattijsen)++ | tools/templates/NQP_REVISION

[09:15] <Geth> ¦ rakudo/main: Bump NQP to get datagram leak fix, Zer0-Tolerance++

[09:15] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/931393bdb9

[09:19] <Geth> ¦ rakudo/main: 6 commits pushed by (Nick Logan)++, (Elizabeth Mattijsen)++

[09:19] <Geth> ¦ rakudo/main: a6b63d6b7c | RakuAST: ask the compiler for its QAST stage rather than probing by name

[09:19] <Geth> ¦ rakudo/main: e5fee93cd6 | RakuAST: add a no-op optimize phase

[09:19] <Geth> ¦ rakudo/main: 801a103e3b | RakuAST: constant-fold pure operators in the optimize phase

[09:19] <Geth> ¦ rakudo/main: 352494749d | RakuAST: collapse a ternary with a constant condition

[09:19] <Geth> ¦ rakudo/main: bd3c6ebb51 | RakuAST: collapse a boolean short-circuit with a constant operand

[09:19] <Geth> ¦ rakudo/main: 4f6b812829 | Merge pull request #6247 from ugexe/ugexe/rakuast-optimizer

[09:19] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/compare/931393bdb93c...4f6b812829d1

[10:01] *** finanalyst joined
[10:49] *** finanalyst left
[11:12] <Geth> ¦ rakudo/main: 5f72bad53c | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod

[11:12] <Geth> ¦ rakudo/main: RakuAST: remove unused attribute

[11:12] <Geth> ¦ rakudo/main: 

[11:12] <Geth> ¦ rakudo/main: The string 'storage-name' didn't occur anywhere in the Rakudo code

[11:12] <Geth> ¦ rakudo/main: base except on this line.  Was originally added with fdafab52bb in

[11:12] <Geth> ¦ rakudo/main: 2021 but appears to have never been used anywhere.

[11:12] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/5f72bad53c

[12:00] <Geth> ¦ rakudo/lizmat-12: 2bce123617 | (Elizabeth Mattijsen)++ | 5 files

[12:00] <Geth> ¦ rakudo/lizmat-12: Another solution to fixing my %h is Set = 1,2,3

[12:00] <Geth> ¦ rakudo/lizmat-12: 

[12:00] <Geth> ¦ rakudo/lizmat-12: This effectively reverts e78c5a7e672786e5ae65 and adds logic to

[12:00] <Geth> ¦ rakudo/lizmat-12: the codegenning to make sure "is Set", "is Bag" and "is Mix" are

[12:00] <Geth> ¦ rakudo/lizmat-12: codegenned with `nqp::create(type)` rather then `type.new`.  This

[12:00] <Geth> ¦ rakudo/lizmat-12: ensures that it will always create a new instance.  Then later

[12:00] <Geth> ¦ rakudo/lizmat-12: the special versions of Set|Bag|Mix.STORE can return the sentinel

[12:00] <Geth> ¦ rakudo/lizmat-12: <…commit message has 8 more lines…>

[12:00] <Geth> ¦ rakudo/lizmat-12: review: https://github.com/rakudo/rakudo/commit/2bce123617

[12:01] <Geth> ¦ rakudo: lizmat++ created pull request #6249: Another solution to fixing my %h is Set = 1,2,3

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

[12:21] *** librasteve_ left
[12:21] *** ab5tract left
[12:21] *** zostay_ joined
[12:21] *** leont left
[12:22] *** zostay left
[12:22] *** zostay_ is now known as zostay

[12:22] *** SmokeMachine left
[12:22] *** tbrowder left
[12:23] *** ab5tract joined
[12:23] *** SmokeMachine joined
[12:24] *** leont joined
[12:25] *** tbrowder joined
[12:36] *** librasteve_ joined
[13:36] <[Coke]>     + Bump NQP to get SQL output refactor fix, timo++ [c46a0f13]

[13:36] <[Coke]> ^^ is this ultimately from moarvm? Don't see a changelog entry in moarvm for it.

[13:36] <[Coke]> ah, no, it's directly from nqp found it

[13:41] <[Coke]> all commits so far in the changelog.

[13:41] <[Coke]> as usually, mostly just a 1:1 from commits, with some merge commits tacked on to one of its included commits.

[13:42] <[Coke]> https://github.com/rakudo/rakudo/wiki/ChangeLog-Draft

[14:10] <ugexe> just to be clear my current intention is to not work on optimizer stuff, but i thought putting some baby optimizations in place would open the door for other people to add more

[14:11] <ugexe> my focus will still generally be on fixing rakuast issues exposed by the ecosystem

[14:32] <[Coke]> ugexe++

[17:04] <Geth> ¦ rakudo/lizmat-13: d135ec9e81 | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod

[17:04] <Geth> ¦ rakudo/lizmat-13: RakuAST: streamline sigil -> role mapping

[17:04] <Geth> ¦ rakudo/lizmat-13: 

[17:04] <Geth> ¦ rakudo/lizmat-13: Make the lookup hash a constant, and only look up the sigil once.

[17:04] <Geth> ¦ rakudo/lizmat-13: 

[17:04] <Geth> ¦ rakudo/lizmat-13: Sadly this is too early to make the hash contain RakuAST::Name

[17:04] <Geth> ¦ rakudo/lizmat-13: objects, so we just keep the name and do the instantiation of

[17:04] <Geth> ¦ rakudo/lizmat-13: RakuAST::Name when needed.

[17:04] <Geth> ¦ rakudo/lizmat-13: review: https://github.com/rakudo/rakudo/commit/d135ec9e81

[17:04] <Geth> ¦ rakudo: lizmat++ created pull request #6250: RakuAST: streamline sigil -> role mapping

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

[17:25] <Geth> ¦ rakudo/lizmat-14: 66f03215af | (Elizabeth Mattijsen)++ | 7 files

[17:25] <Geth> ¦ rakudo/lizmat-14: RakuAST: make some lookup hashes set at compile time

[17:25] <Geth> ¦ rakudo/lizmat-14: 

[17:25] <Geth> ¦ rakudo/lizmat-14: So they don't need to be re-created and initialized on each call.

[17:25] <Geth> ¦ rakudo/lizmat-14: review: https://github.com/rakudo/rakudo/commit/66f03215af

[17:27] <Geth> ¦ rakudo: lizmat++ created pull request #6251: RakuAST: make some lookup hashes set at compile time

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

[17:27] <lizmat> argh... that also included lizmat-13

[17:27] <lizmat> ah well

[17:27] <lizmat> it was just for the RakuAST CI tests really

[17:34] <Geth> ¦ rakudo/main: 336e76501a | (Nick Logan)++ (committed using GitHub Web editor) | 3 files

[17:34] <Geth> ¦ rakudo/main: RakuAST: give onlystar bodies a source location

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

[17:34] <Geth> ¦ rakudo/main: A proto's onlystar body compiles to a frame whose only code is the

[17:34] <Geth> ¦ rakudo/main: dispatch instruction, and that frame carried no location annotation.

[17:34] <Geth> ¦ rakudo/main: The OnlyStar node is created directly in the routine actions rather

[17:34] <Geth> ¦ rakudo/main: than through an action method of its own, so it never received an

[17:34] <Geth> ¦ rakudo/main: origin, and its QAST carried no node to annotate from. Code.file and

[17:34] <Geth> ¦ rakudo/main: <…commit message has 16 more lines…>

[17:34] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/336e76501a

[17:39] <Geth> ¦ rakudo/main: 0918c0552e | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 5 files

[17:39] <Geth> ¦ rakudo/main: Another solution to fixing my %h is Set = 1,2,3

[17:39] <Geth> ¦ rakudo/main: 

[17:39] <Geth> ¦ rakudo/main: This effectively reverts e78c5a7e672786e5ae65 and adds logic to

[17:39] <Geth> ¦ rakudo/main: the codegenning to make sure "is Set", "is Bag" and "is Mix" are

[17:39] <Geth> ¦ rakudo/main: codegenned with `nqp::create(type)` rather then `type.new`.  This

[17:39] <Geth> ¦ rakudo/main: ensures that it will always create a new instance.  Then later

[17:39] <Geth> ¦ rakudo/main: the special versions of Set|Bag|Mix.STORE can return the sentinel

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

[17:39] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/0918c0552e

[17:49] <Geth> ¦ roast: 26fc8d9736 | (Elizabeth Mattijsen)++ | 3 files

[17:49] <Geth> ¦ roast: Add tests for "is Set|Bag|Mix" keeping sentinels clean

[17:49] <Geth> ¦ roast: 

[17:49] <Geth> ¦ roast: See https://github.com/rakudo/rakudo/issues/6246

[17:49] <Geth> ¦ roast: review: https://github.com/Raku/roast/commit/26fc8d9736

[17:53] <Geth> ¦ rakudo/main: 69b2592291 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | src/core.c/Array.rakumod

[17:53] <Geth> ¦ rakudo/main: Fix Array.shift on holes in arrays

[17:53] <Geth> ¦ rakudo/main: 

[17:53] <Geth> ¦ rakudo/main: Unlike the needed Array.pop fix in 415fa4b069 to prevent a segfault,

[17:53] <Geth> ¦ rakudo/main: Array.shift was already guarded against returning an nqp::null.

[17:53] <Geth> ¦ rakudo/main: 

[17:53] <Geth> ¦ rakudo/main: However, it was set to return Nil, rather than the default value

[17:53] <Geth> ¦ rakudo/main: of the array.  This commit changes that to be more in line with

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

[17:53] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/69b2592291

[18:03] <Geth> ¦ roast: b87c37fc1c | (Elizabeth Mattijsen)++ | S32-array/shift.t

[18:03] <Geth> ¦ roast: Fix some .shift tests

[18:03] <Geth> ¦ roast: 

[18:03] <Geth> ¦ roast: 1. https://github.com/rakudo/rakudo/pull/6242 changed the behaviour

[18:03] <Geth> ¦ roast:    of .shift on a hole to returning the default value of the array

[18:03] <Geth> ¦ roast:    rather then Nil (also what one would get if one would do @a[0]

[18:03] <Geth> ¦ roast:    on a hole)

[18:03] <Geth> ¦ roast: 

[18:03] <Geth> ¦ roast: 2. Activate test of shifting off an infinite array: that apparently

[18:03] <Geth> ¦ roast:    works nowadays :-)

[18:03] <Geth> ¦ roast: review: https://github.com/Raku/roast/commit/b87c37fc1c

[18:11] <Geth> ¦ rakudo/main: b0c6b1b9f8 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 8 files

[18:11] <Geth> ¦ rakudo/main: RakuAST: make some lookup hashes set at compile time

[18:11] <Geth> ¦ rakudo/main: 

[18:11] <Geth> ¦ rakudo/main: So they don't need to be re-created and initialized on each call.

[18:11] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/b0c6b1b9f8

[18:23] *** hurufu1 joined
[18:23] *** hurufu left
[18:23] *** hurufu1 is now known as hurufu

[18:58] <Geth> ¦ nqp/main: 04a224de3e | (Nick Logan)++ (committed using GitHub Web editor) | 4 files

[18:58] <Geth> ¦ nqp/main: HLL::Compiler.compile: balance backend per-compile state on early exit

[18:58] <Geth> ¦ nqp/main: 

[18:58] <Geth> ¦ nqp/main: HLL::Backend.start (MoarVM) pushes a frame list onto

[18:58] <Geth> ¦ nqp/main: %COMPILING<moar><frames> at every compile's 'start' stage. The matching

[18:58] <Geth> ¦ nqp/main: pop runs at the 'mast' stage in QASTCompilerMAST.to_mast. A compile

[18:58] <Geth> ¦ nqp/main: that stops before 'mast' (e.g. with `:target('ast')`), or one whose

[18:58] <Geth> ¦ nqp/main: stage call throws, leaves the push dangling. The next nested compile's

[18:58] <Geth> ¦ nqp/main: <…commit message has 16 more lines…>

[18:58] <Geth> ¦ nqp/main: review: https://github.com/Raku/nqp/commit/04a224de3e

[19:08] <Geth> ¦ rakudo/main: 25200ba632 | (Elizabeth Mattijsen)++ | tools/templates/NQP_REVISION

[19:08] <Geth> ¦ rakudo/main: Bump NQP to get balanced backend state on early exit

[19:08] <Geth> ¦ rakudo/main: 

[19:08] <Geth> ¦ rakudo/main: ugexe++

[19:08] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/25200ba632

[20:09] <lizmat> m: say nqp::const::CCLASS_ANY  # apparently, nqp::const::xxx does *not* need "use nqp" 

[20:09] <camelia> rakudo-moar 25200ba63: OUTPUT: «65535␤»

[20:09] <lizmat> TIL

[20:10] *** finanalyst joined
[20:17] <[Coke]> that seems odd

[20:57] <Geth> ¦ rakudo/lizmat-15: 6ef106b169 | (Elizabeth Mattijsen)++ | src/Raku/Actions.nqp

[20:57] <Geth> ¦ rakudo/lizmat-15: RakuAST: some streamlining in actions

[20:57] <Geth> ¦ rakudo/lizmat-15: 

[20:57] <Geth> ¦ rakudo/lizmat-15: 1. prevent repeated named capture lookups

[20:57] <Geth> ¦ rakudo/lizmat-15: 2. streamline is-yada to not use a regular expression, but more

[20:57] <Geth> ¦ rakudo/lizmat-15:    native logic to check if a package is a stub

[20:57] <Geth> ¦ rakudo/lizmat-15: review: https://github.com/rakudo/rakudo/commit/6ef106b169

[20:57] <Geth> ¦ rakudo: lizmat++ created pull request #6252: RakuAST: some streamlining in actions

[20:57] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6252

[21:52] *** hurufu left
[22:03] *** finanalyst left