[00:02] *** lucs_ is now known as lucs

[05:41] *** kjp left
[05:46] *** kjp joined
[07:04] <Geth> ¦ rakudo/lizmat-59: 81ae3f8048 | (Elizabeth Mattijsen)++ | src/Raku/ast/expressions.rakumod

[07:04] <Geth> ¦ rakudo/lizmat-59: RakuAST: simplify ::ApplyPrefix.IMPL-RECORD-NATIVE-RETURN-TYPE

[07:04] <Geth> ¦ rakudo/lizmat-59: 

[07:04] <Geth> ¦ rakudo/lizmat-59: Since it didn't matter why the $routine value could not produce

[07:04] <Geth> ¦ rakudo/lizmat-59: a concrete value, wrap all of the checks in a single try.

[07:04] <Geth> ¦ rakudo/lizmat-59: 

[07:04] <Geth> ¦ rakudo/lizmat-59: Also use a proper if structure for:

[07:04] <Geth> ¦ rakudo/lizmat-59: 1. readability

[07:04] <Geth> ¦ rakudo/lizmat-59: 2. no need for return statements

[07:04] <Geth> ¦ rakudo/lizmat-59: review: https://github.com/rakudo/rakudo/commit/81ae3f8048

[07:04] <Geth> ¦ rakudo/lizmat-59: ecdde5fc3d | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 17 files

[07:04] <Geth> ¦ rakudo/lizmat-59: Merge branch 'main' into lizmat-59

[07:04] <Geth> ¦ rakudo/lizmat-59: review: https://github.com/rakudo/rakudo/commit/ecdde5fc3d

[07:04] <lizmat> :q

[07:07] <Geth> ¦ rakudo/lizmat-59: 1cb953b3b9 | (Elizabeth Mattijsen)++ | src/Raku/ast/expressions.rakumod

[07:07] <Geth> ¦ rakudo/lizmat-59: RakuAST: add condition that was missed

[07:07] <Geth> ¦ rakudo/lizmat-59: review: https://github.com/rakudo/rakudo/commit/1cb953b3b9

[07:19] <Geth> ¦ rakudo/lizmat-60: 30f89a24c9 | (Elizabeth Mattijsen)++ | 3 files

[07:19] <Geth> ¦ rakudo/lizmat-60: RakuAST: create ::IfUnless base class

[07:19] <Geth> ¦ rakudo/lizmat-60: 

[07:19] <Geth> ¦ rakudo/lizmat-60: The handling of RakuAST::StatementModifier::If and ::Unless differ

[07:19] <Geth> ¦ rakudo/lizmat-60: only in the nqp op that generates the QAST.  Instead ::Unless doing

[07:19] <Geth> ¦ rakudo/lizmat-60: a cat-license of ::If, create a base class ::IfUnless referring to

[07:19] <Geth> ¦ rakudo/lizmat-60: a method that provides the op-name, and make ::If and ::Unless

[07:19] <Geth> ¦ rakudo/lizmat-60: provide the correct op name.

[07:19] <Geth> ¦ rakudo/lizmat-60: 

[07:19] <Geth> ¦ rakudo/lizmat-60: This also allows some conditions /base and /var-loweing to be

[07:19] <Geth> ¦ rakudo/lizmat-60: simplified.

[07:19] <Geth> ¦ rakudo/lizmat-60: review: https://github.com/rakudo/rakudo/commit/30f89a24c9

[07:19] <Geth> ¦ rakudo: lizmat++ created pull request #6831: RakuAST: create ::IfUnless base class

[07:19] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6831

[08:37] *** finanalyst left
[08:58] <Geth> ¦ rakudo/lizmat-61: ca3f4b86fe | (Elizabeth Mattijsen)++ | 4 files

[08:58] <Geth> ¦ rakudo/lizmat-61: RakuAST: nqp::objprimspec can handle nqp::null

[08:58] <Geth> ¦ rakudo/lizmat-61: 

[08:58] <Geth> ¦ rakudo/lizmat-61: So there is no need to extra guard against this eventuality.

[08:58] <Geth> ¦ rakudo/lizmat-61: 

[08:58] <Geth> ¦ rakudo/lizmat-61: Also slightly re-arrange some if's into ternaries

[08:58] <Geth> ¦ rakudo/lizmat-61: review: https://github.com/rakudo/rakudo/commit/ca3f4b86fe

[09:08] <Geth> ¦ roast: 2ee2f513d2 | (Elizabeth Mattijsen)++ | S13-overloading/operators.t

[09:08] <Geth> ¦ roast: Add test for https://github.com/rakudo/rakudo/pull/6809

[09:08] <Geth> ¦ roast: review: https://github.com/Raku/roast/commit/2ee2f513d2

[09:32] <Geth> ¦ rakudo: lizmat++ created pull request #6832: RakuAST: nqp::objprimspec can handle nqp::null

[09:32] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6832

[11:18] <Geth> ¦ roast: 7b27845222 | (Elizabeth Mattijsen)++ | S13-overloading/operators.t

[11:18] <Geth> ¦ roast: Skip test, mark as fixed in RakuAST

[11:18] <Geth> ¦ roast: review: https://github.com/Raku/roast/commit/7b27845222

[11:32] <lizmat> m: my class A does Callable { has Str $.signature = "foo"; method CALL-ME($b) { "$b $!signature" } }; my constant &prefix:<c> = A.new; say c "hello";

[11:32] <camelia> rakudo-moar e17a1d43b: OUTPUT: «===SORRY!===␤This representation (P6str) does not support elems (for type str)␤»

[11:32] <lizmat> bisectable6: old=2025.01 my class A does Callable { has Str $.signature = "foo"; method CALL-ME($b) { "$b $!signature" } }; my constant &prefix:<c> = A.new; say c "hello";

[11:32] <bisectable6> lizmat, Bisecting by exit code (old=2025.01 new=0cdbad0). Old exit code: 1

[11:33] <bisectable6> lizmat, bisect log: https://gist.github.com/3c288afb75a2787617b6085a8fb4623a

[11:33] <bisectable6> lizmat, (2026-09-22) https://github.com/rakudo/rakudo/commit/f0a24041462ebd6979a8648ccb2954b3348dbcb0

[11:33] <lizmat> c: HEAD  my class A does Callable { has Str $.signature = "foo"; method CALL-ME($b) { "$b $!signature" } }; my constant &prefix:<c> = A.new; say c "hello";

[11:33] <committable6> lizmat, ¦HEAD(0cdbad0): «hello foo␤»

[11:35] <lizmat> bisectable6: old=2025.01 new=2026.08 my class A does Callable { has Str $.signature = "foo"; method CALL-ME($b) { "$b $!signature" } }; my constant &prefix:<c> = A.new; say c "hello";

[11:35] <bisectable6> lizmat, On both starting points (old=2025.01 new=2026.08) the exit code is 1 and the output is identical as well

[11:35] <bisectable6> lizmat, Output on both points: «===SORRY!===␤This representation (P6str) does not support elems (for type str)␤»

[11:35] <lizmat> bisectable6: new=2026.08 my class A does Callable { has Str $.signature = "foo"; method CALL-ME($b) { "$b $!signature" } }; my constant &prefix:<c> = A.new; say c "hello";

[11:35] <bisectable6> lizmat, Bisecting by output (old=2015.12 new=2026.08) because on both starting points the exit code is 1

[11:36] <bisectable6> lizmat, bisect log: https://gist.github.com/5c34ab485cabb866820c7025dfe47df6

[11:36] <bisectable6> lizmat, (2016-05-12) https://github.com/rakudo/rakudo/commit/33ef5a95459b95bf7256391a053550d4efe221de

[11:46] <lizmat> looking at the bisect log, it looks like it has been in there since Christmas

[15:30] <Geth> ¦ nqp/main: 5e40020521 | (Elizabeth Mattijsen)++ | tools/templates/MOAR_REVISION

[15:30] <Geth> ¦ nqp/main: Bump MoarVM for JITted ops fix, ugexe++

[15:30] <Geth> ¦ nqp/main: review: https://github.com/Raku/nqp/commit/5e40020521

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

[15:43] <Geth> ¦ rakudo/main: Bump NQP for JITted ops fix, ugexe++

[15:43] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/d4d0df63cc

[15:48] <Geth> ¦ nqp/main: 13a9fad534 | (Elizabeth Mattijsen)++ | tools/templates/MOAR_REVISION

[15:48] <Geth> ¦ nqp/main: Bump MoarVM to fix an issue with spesh, ugexe++

[15:48] <Geth> ¦ nqp/main: review: https://github.com/Raku/nqp/commit/13a9fad534

[16:13] <Geth> ¦ rakudo/lizmat-62: 3388d045c2 | (Elizabeth Mattijsen)++ | tools/templates/NQP_REVISION

[16:13] <Geth> ¦ rakudo/lizmat-62: Bump NQP to fix an issue with spesh, ugexe++

[16:13] <Geth> ¦ rakudo/lizmat-62: review: https://github.com/rakudo/rakudo/commit/3388d045c2

[16:13] <Geth> ¦ rakudo/lizmat-62: e0ddce3c59 | (Elizabeth Mattijsen)++ | 5 files

[16:13] <Geth> ¦ rakudo/lizmat-62: RakuAST: remove unnecessary nqp::bindattr_i() calls

[16:13] <Geth> ¦ rakudo/lizmat-62: 

[16:13] <Geth> ¦ rakudo/lizmat-62: At object creation, any native integer attributes are created with

[16:13] <Geth> ¦ rakudo/lizmat-62: the value 0, so there is no need to actually bind the value 0 to

[16:13] <Geth> ¦ rakudo/lizmat-62: them.

[16:13] <Geth> ¦ rakudo/lizmat-62: review: https://github.com/rakudo/rakudo/commit/e0ddce3c59

[16:14] <Geth> ¦ rakudo: lizmat++ created pull request #6833: Lizmat 62

[16:14] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6833

[16:14] <lizmat> argh

[16:16] <Geth> ¦ rakudo/main: 3388d045c2 | (Elizabeth Mattijsen)++ | tools/templates/NQP_REVISION

[16:16] <Geth> ¦ rakudo/main: Bump NQP to fix an issue with spesh, ugexe++

[16:16] <Geth> ¦ rakudo/main: review: https://github.com/rakudo/rakudo/commit/3388d045c2

[16:25] *** MasterDuke joined
[16:28] <MasterDuke> ugexe: there are now a *ton* of warnings when compiling MoarVM: `src/jit/graph.c:4119:50: warning: missing initializer for field ‘native_idx’ of ‘MVMJitCallArg’ [-Wmissing-field-initializers]

[16:28] <MasterDuke>  4119 |                                                  { MVM_JIT_DATA_LABEL,  { data_label } }};

[16:28] <MasterDuke>       |                                                  ^

[16:28] <MasterDuke> src/jit/graph.h:143:14: note: ‘native_idx’ declared here

[16:28] <MasterDuke>   143 |     MVMint16 native_idx;

[16:28] <MasterDuke>       |              ^~~~~~~~~~`

[16:28] <MasterDuke> i didn't notice them at first when building on my laptop, but only on my desktop

[16:29] * lizmat checks

[16:29] <MasterDuke> adding something to the MVMJitCallArg struct means a lot of uses need to change. can the fix be done without needing to add native_idx to MVMJitCallArg?

[16:31] <lizmat> MasterDuke: hmm.. your approval was a sign for me to merge, should I rollback ?

[16:32] <MasterDuke> i wouldn't bother just yet. we don't have a release imminent, there's time to see if it can be cleared up without requiring a revert

[16:32] <lizmat> ack

[16:32] <lizmat> fwiw, I didn't see any warnings on MacOS, but that all makes sense as it doesn't have a JIT

[16:32] <lizmat> :-(

[17:51] *** MasterDuke left
[18:22] <Geth> ¦ rakudo/lizmat-63: 761c2e7d74 | (Elizabeth Mattijsen)++ | src/Raku/ast/expressions.rakumod

[18:22] <Geth> ¦ rakudo/lizmat-63: RakuAST: add base class RakuAST::MetaInfix::CrossZip

[18:22] <Geth> ¦ rakudo/lizmat-63: 

[18:22] <Geth> ¦ rakudo/lizmat-63: The logic of RakuAST::MetaInfix::Cross and RakuAST::MetaInfix::Zip

[18:22] <Geth> ¦ rakudo/lizmat-63: are identical, except for three values:

[18:22] <Geth> ¦ rakudo/lizmat-63: 

[18:22] <Geth> ¦ rakudo/lizmat-63: - action ('cross with' | 'zip with')

[18:22] <Geth> ¦ rakudo/lizmat-63: - metaop ('X' | 'Z')

[18:22] <Geth> ¦ rakudo/lizmat-63: <…commit message has 11 more lines…>

[18:22] <Geth> ¦ rakudo/lizmat-63: review: https://github.com/rakudo/rakudo/commit/761c2e7d74

[18:23] <Geth> ¦ rakudo: lizmat++ created pull request #6834: RakuAST: add base class RakuAST::MetaInfix::CrossZip

[18:23] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6834

[18:48] <Geth> ¦ rakudo/lizmat-64: 78784d4d18 | (Elizabeth Mattijsen)++ | 12 files

[18:48] <Geth> ¦ rakudo/lizmat-64: RakuAST: native int variables don't need := 0 initialization

[18:48] <Geth> ¦ rakudo/lizmat-64: 

[18:48] <Geth> ¦ rakudo/lizmat-64: Native int variables are "born" with the value 0.

[18:48] <Geth> ¦ rakudo/lizmat-64: 

[18:48] <Geth> ¦ rakudo/lizmat-64: This removes this unnecessary initialization from many lexical

[18:48] <Geth> ¦ rakudo/lizmat-64: variables, and also changes some variables to native ints as they're

[18:48] <Geth> ¦ rakudo/lizmat-64: clearly intended to be used as flags.

[18:48] <Geth> ¦ rakudo/lizmat-64: review: https://github.com/rakudo/rakudo/commit/78784d4d18

[18:49] <Geth> ¦ rakudo: lizmat++ created pull request #6835: RakuAST: native int variables don't need := 0 initialization

[18:49] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6835

[19:09] <Geth> ¦ rakudo/lizmat-65: e73567213f | (Elizabeth Mattijsen)++ | 3 files

[19:09] <Geth> ¦ rakudo/lizmat-65: RakuAST: make several lookup tables compile-time

[19:09] <Geth> ¦ rakudo/lizmat-65: 

[19:09] <Geth> ¦ rakudo/lizmat-65: So that they don't need to be initialized every time they are about

[19:09] <Geth> ¦ rakudo/lizmat-65: to be accessed.

[19:09] <Geth> ¦ rakudo/lizmat-65: review: https://github.com/rakudo/rakudo/commit/e73567213f

[19:10] <Geth> ¦ rakudo: lizmat++ created pull request #6836: RakuAST: make several lookup tables compile-time

[19:10] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6836

[19:20] <Geth> ¦ rakudo/lizmat-66: c3a443cb9d | (Elizabeth Mattijsen)++ | src/Raku/ast/expressions.rakumod

[19:20] <Geth> ¦ rakudo/lizmat-66: RakuAST: replace variable by literal

[19:20] <Geth> ¦ rakudo/lizmat-66: 

[19:20] <Geth> ¦ rakudo/lizmat-66: This variable was only initialized to zero, and then used without

[19:20] <Geth> ¦ rakudo/lizmat-66: being referenced or mutated anywhere else.

[19:20] <Geth> ¦ rakudo/lizmat-66: review: https://github.com/rakudo/rakudo/commit/c3a443cb9d

[19:21] <Geth> ¦ rakudo: lizmat++ created pull request #6837: RakuAST: replace variable by literal

[19:21] <Geth> ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/6837

[20:00] <Geth> ¦ rakudo/lizmat-67: b9507910c7 | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod

[20:00] <Geth> ¦ rakudo/lizmat-67: RakuAST: simplify missing initializer test

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

[20:00] <Geth> ¦ rakudo/lizmat-67: No need to create intermediate variables

[20:00] <Geth> ¦ rakudo/lizmat-67: review: https://github.com/rakudo/rakudo/commit/b9507910c7

[20:00] <Geth> ¦ rakudo: lizmat++ created pull request #6838: RakuAST: simplify missing initializer test

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

[20:06] <[Coke]> c: 3388d045c2 e

[20:06] <committable6> [Coke], ¦3388d04: «»

[20:35] <Geth> ¦ rakudo/lizmat-68: e9359055c4 | (Elizabeth Mattijsen)++ | src/Raku/ast/variable-declaration.rakumod

[20:35] <Geth> ¦ rakudo/lizmat-68: RakuAST: default native initialize only for strings

[20:35] <Geth> ¦ rakudo/lizmat-68: 

[20:35] <Geth> ¦ rakudo/lizmat-68: Native variables already initialize automatically to their base

[20:35] <Geth> ¦ rakudo/lizmat-68: value, so don't need to have any QAST added for that.

[20:35] <Geth> ¦ rakudo/lizmat-68: review: https://github.com/rakudo/rakudo/commit/e9359055c4

[20:36] <Geth> ¦ rakudo: lizmat++ created pull request #6839: RakuAST: default native initialize only for strings

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

[22:16] *** vrurg_ joined
[22:17] *** vrurg left