[00:02] *** b2gills joined [00:19] *** chansen_ joined [00:35] *** Kaeipi left [00:35] *** Kaeipi joined [00:43] *** dogbert17 left [01:29] *** kvw_5_ joined [01:32] *** kvw_5 left [01:33] *** dogbert17 joined [05:04] *** Geth joined [06:31] *** squashable6 left [06:32] *** squashable6 joined [06:39] ¦ nqp: e4c8a5c30e | (Patrick Böker)++ | README.pod [06:39] ¦ nqp: Remove Travis badge [06:39] ¦ nqp: [06:39] ¦ nqp: Travis has been disabled since the beginning of January already. [06:39] ¦ nqp: review: https://github.com/Raku/nqp/commit/e4c8a5c30e [06:39] ¦ nqp: d88a3e4c25 | (Patrick Böker)++ (committed using GitHub Web editor) | README.pod [06:39] ¦ nqp: Merge pull request #707 from patrickbkr/disable-obsolete-CIs [06:39] ¦ nqp: [06:39] ¦ nqp: Remove Travis badge [06:39] ¦ nqp: review: https://github.com/Raku/nqp/commit/d88a3e4c25 [06:43] *** domidumont joined [06:51] <[Tux]> Rakudo v2021.03-47-g8fc4cb56c (v6.d) on MoarVM 2021.03-16-gd7220f95a [06:51] <[Tux]> csv-test-xs-20 0.367 - 0.370 [06:51] <[Tux]> csv-ip5xs 0.828 - 0.832 [06:51] <[Tux]> test-t --race 0.863 - 0.983 [06:51] <[Tux]> test-t 1.892 - 1.954 [06:51] <[Tux]> test 7.727 - 7.729 [06:51] <[Tux]> csv-ip5xs-20 8.053 - 8.446 [06:51] <[Tux]> test-t-20 --race 9.041 - 9.414 [06:51] <[Tux]> csv-parser 25.002 - 25.687 [06:51] <[Tux]> test-t-20 33.498 - 34.265 [06:56] *** HarmtH left [06:57] *** HarmtH joined [06:59] *** patrickb joined [06:59] o/ [07:09] *** frost-lab joined [07:40] *** linkable6 left [07:41] *** linkable6 joined [07:56] *** Kaeipi left [07:58] *** Kaiepi joined [08:54] *** linkable6 left [08:55] *** linkable6 joined [09:09] very interesting. https://github.com/Raku/nqp/blob/master/src/NQP/Compiler.nqp#L44-L53 has the same `#?if moar ... ` business with `@ARGS` vs `*@ARGS` that https://github.com/rakudo/rakudo/blob/master/src/main.nqp and https://github.com/rakudo/rakudo/blob/master/src/rakudo-debug.nqp do [09:10] and easier to experiment with since i don't get that circular module error when trying to run multiple times [09:11] and it's been that way for 8 years [09:12] if i change the `#?if moar` version to match the others, then the nqp build seems to drop into a repl, where ^D starts the next step in the build [09:14] there were mentions of parrot with an 'XXX FIX ME' in previous versions, so i have no idea what's the correct behavior [09:35] *** linkable6 left [09:36] *** linkable6 joined [09:38] m: my $a = "»ö«".uniname for ^10_000_000; say now - INIT now [09:38] rakudo-moar 726a75e24: OUTPUT: «1.2926607␤» [09:38] m: my $a = uniname("»ö«") for ^10_000_000; say now - INIT now [09:38] rakudo-moar 726a75e24: OUTPUT: «1.4239268␤» [09:38] m: my $a = uniname("»ö«") for ^10_000_000; say now - INIT now [09:38] rakudo-moar 726a75e24: OUTPUT: «1.36861578␤» [09:39] heh, the sub version is a lot faster on my home machine [09:40] how old is camelia? [09:43] feb 24 [09:43] dogbert17: the sub version is currently faster as it doesn't dispatch to the method version [09:43] this will be corrected in the coming days [09:43] ++lizmat, I had a feeling it was on your radar :) [09:43] so what can we do about camelia then? [09:47] I'm not sure who runs camelia at the moment [09:47] could be nine, could be moritz_ ? [09:50] ¦ rakudo: 8f4db8b6b8 | (Elizabeth Mattijsen)++ | src/core.c/List.pm6 [09:50] ¦ rakudo: Make List.sum again 40% faster [09:50] ¦ rakudo: [09:50] ¦ rakudo: By skipping the special handling of the first element. This also [09:50] ¦ rakudo: reduces the bytecode size, but still too large to get inlined. However [09:50] ¦ rakudo: all of the method calls inside "sum" *are* inlined into "sum". [09:50] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/8f4db8b6b8 [10:21] *** gfldex_ is now known as gfldex [10:29] ¦ rakudo: 07aa4aea9f | (Elizabeth Mattijsen)++ | 2 files [10:29] ¦ rakudo: Make Any.sum about 40% faster and check for laziness [10:29] ¦ rakudo: [10:29] ¦ rakudo: - move it to Any-iterable, as it is more a place to be for such a function [10:29] ¦ rakudo: - use binding instead of assignment [10:29] ¦ rakudo: - add private method !lazy for handling of lazy iterators [10:29] ¦ rakudo: - reduce bytecode size by handling laziness outside in !lazy [10:29] ¦ rakudo: - Any.sum now small enough in bytesize to inline [10:29] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/07aa4aea9f [10:40] *** linkable6 left [10:41] *** linkable6 joined [10:54] ¦ rakudo: e191266f88 | (Elizabeth Mattijsen)++ | 6 files [10:54] ¦ rakudo: Make .sum a proper multi [10:54] ¦ rakudo: [10:54] ¦ rakudo: - put the "is nodal" trait on the multi [10:54] ¦ rakudo: - all candidates now are "nodal" because of that [10:54] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/e191266f88 [11:01] *** frost-lab left [11:01] *** frost-lab joined [11:32] *** frost-lab left [11:37] ¦ rakudo: 8e56c8c89f | (Elizabeth Mattijsen)++ | 2 files [11:37] ¦ rakudo: Introduce (throw|fail)-iterator-cannot-be-lazy methods [11:37] ¦ rakudo: [11:37] ¦ rakudo: - marked as implementation-detail [11:37] ¦ rakudo: - utility method to fail / throw X::Cannot::Lazy exception [11:37] ¦ rakudo: - to be used by methods not accepting lazy iterators [11:37] ¦ rakudo: - intended to reduce bytecode size of calles to help inlineability [11:37] ¦ rakudo: - removed the !lazy private methods that were used previously [11:37] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/8e56c8c89f [12:17] *** sena_kun left [12:17] *** sena_kun joined [14:19] *** patrickb left [15:07] ¦ rakudo: tbrowder self-assigned A DateTime object should output a real number for its posix method https://github.com/rakudo/rakudo/issues/4286 [16:39] ¦ rakudo/master: 14 commits pushed by (Elizabeth Mattijsen)++ [16:39] ¦ rakudo/master: review: https://github.com/rakudo/rakudo/compare/8e56c8c89f3d...9d1b482185b0 [17:14] ¦ rakudo: ac9f40d871 | (Elizabeth Mattijsen)++ | 8 files [17:14] ¦ rakudo: Introduce (fail|throw)-cannot-be-empty helper method [17:14] ¦ rakudo: [17:14] ¦ rakudo: To again reduce bytecode size in the caller's for better inlineabiolity [17:14] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/ac9f40d871 [17:14] *** domidumont left [17:17] ¦ nqp: usev6++ created pull request #708: [JVM] Add op 'nativecallinvoke' [17:17] ¦ nqp: review: https://github.com/Raku/nqp/pull/708 [17:37] ¦ rakudo: 71960b162d | (Elizabeth Mattijsen)++ | 20 files [17:37] ¦ rakudo: Set more sensible defaults to helper methods [17:37] ¦ rakudo: [17:37] ¦ rakudo: Further reduces bytecode footprint at the callsites [17:37] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/71960b162d [17:40] ¦ rakudo: usev6++ created pull request #4287: Remove unused variable [17:40] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/4287 [17:40] *** MasterDuke left [17:57] does anyone know how to find the name or alias of a github emoji submitter on a comment in PRs or issues? [18:47] *** HarmtH left [18:50] *** HarmtH joined [18:58] ¦ rakudo: b0910d5bd1 | (Christian Bartolomäus)++ | lib/NativeCall.rakumod [18:58] ¦ rakudo: Remove unused variable [18:58] ¦ rakudo: [18:58] ¦ rakudo: It was added with 80d6b425ce and it looks like a leftover from [18:58] ¦ rakudo: developing that patch. (Maybe it was meant to be passed as the [18:58] ¦ rakudo: value for :deserialize_ast -- which used $fixups in the end.) [18:58] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/b0910d5bd1 [19:18] *** HarmtH_ joined [19:18] *** HarmtH left [19:23] ¦ rakudo: 23036f1d7a | (Elizabeth Mattijsen)++ | src/core.c/Any-iterable-methods.pm6 [19:23] ¦ rakudo: Save one scalar allocation for each non-simple for loop [19:23] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/23036f1d7a [19:48] *** HarmtH_ left [19:50] *** HarmtH joined [20:28] ¦ rakudo: f75957dd8e | (Elizabeth Mattijsen)++ | 3 files [20:28] ¦ rakudo: Make .chrs a proper multi [20:28] ¦ rakudo: [20:28] ¦ rakudo: Also add a native int candidate, which is about 30% faster. [20:28] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/f75957dd8e [20:45] *** MasterDuke joined [20:45] *** dogbert11 joined [20:49] *** dogbert17 left [21:05] ¦ nqp-configure: Tyil++ created pull request #26: Consider /dragonfly/ a BSD as well [21:05] ¦ nqp-configure: review: https://github.com/Raku/nqp-configure/pull/26 [21:08] tbrowder: hover over the emoji [21:24] ¦ rakudo: 31331a2861 | (Elizabeth Mattijsen)++ | src/core.c/unicodey.pm6 [21:24] ¦ rakudo: Make uniname a proper multi method [21:24] ¦ rakudo: [21:24] ¦ rakudo: - move the main logic to the Int / Str classes [21:24] ¦ rakudo: - make the sub version just pass on to the appropriate method [21:24] ¦ rakudo: - add support for other Cool values such as Num and Match [21:24] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/31331a2861 [21:25] ¦ nqp-configure: e6ddc07aa9 | (Patrick Spek)++ | lib/NQP/Config.pm [21:25] ¦ nqp-configure: Consider /dragonfly/ a BSD as well [21:25] ¦ nqp-configure: [21:25] ¦ nqp-configure: In Perl, $^O is dragonfly when running DragonflyBSD. As this doesn't [21:25] ¦ nqp-configure: match the /bsd/ that is done to determine bsd-ness, this screws up the [21:25] ¦ nqp-configure: `make` check, which results in NQP and Rakudo failing to build on [21:25] ¦ nqp-configure: DragonflyBSD. [21:25] ¦ nqp-configure: review: https://github.com/Raku/nqp-configure/commit/e6ddc07aa9 [21:25] ¦ nqp-configure: e068508a94 | (Vadim Belman)++ (committed using GitHub Web editor) | lib/NQP/Config.pm [21:25] ¦ nqp-configure: Merge pull request #26 from Tyil/dragonflybsd [21:25] ¦ nqp-configure: [21:25] ¦ nqp-configure: Consider /dragonfly/ a BSD as well [21:25] ¦ nqp-configure: review: https://github.com/Raku/nqp-configure/commit/e068508a94 [21:25] \o/ [21:26] can anyone tell me when I can expect NQP/Rakudo to use this latest update to nqp-configure? [21:26] (it's currently breaking rakudo-star for a user, and I want to make them happy with a nightly that they can use) [21:28] ¦ rakudo: 80d2034e13 | (Vadim Belman)++ | 3rdparty/nqp-configure [21:28] ¦ rakudo: nqp-configure submodule update [21:28] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/80d2034e13 [21:29] tyil: was it the answer? ;) [21:30] :D [21:30] that's way faster than I expected, not gonna lie [21:36] moon-child: thnx, but that doesn't work on an ipad so far (unless there's some finger magic i don't know about) [21:43] oh, heh, don't know then [21:52] vrurg: I clones latest moarvm, nqp and rakudo for a quick rakudo-star testing drive, but nqp doesn't want to build: `Found a MoarVM binary but was not able to get its version number.` [21:52] I'm going to guess it's an easy thing to fix, but I wouldn't know where to look :< [21:52] it says I *should* check out tags, but then I won't have the fix I want [21:53] (if it's a chore to do it like this I'll just patiently wait for 2021.4, though) [21:55] ¦ nqp: cb008e907a | (Vadim Belman)++ | 3rdparty/nqp-configure [21:55] ¦ nqp: nqp-configure submodule update [21:55] ¦ nqp: review: https://github.com/Raku/nqp/commit/cb008e907a [21:55] tyil: ./Config --gen-nqp=master [21:55] Config.pl [21:56] ah, thanks :> [21:57] I have forgotten to update the submodule in nqp, but now it's fixed. Not sure if MoarVM is ready for dragonfly though. [22:09] moarvm builds fine on its own, at least [22:10] --gen-nqp is not a valid flag for NQP's Configure.pl it looks like [22:11] (fwiw, I have each moarvm,nqp and rakudo checked out individually, and I build each part from its own repo in rstar) [22:11] tyil: It's for rakudo build to get the latest NQP. [22:11] hmm [22:12] tyil: Since you build separately, you don't need --gen-nqp then. Just update NQP report to the latest master. [22:12] s/report/repo/ [22:13] I already pulled latest master on nqp [22:15] it is late, perhaps I'm doing something dumb, I should retry in the morning [22:15] vrurg: thanks for your help thus far :> [22:16] tyil: np and good luck! One more platform is good to have. [22:19] ¦ rakudo: d14dda68da | (Elizabeth Mattijsen)++ | src/core.c/unicodey.pm6 [22:19] ¦ rakudo: Make uninames a proper multi method [22:19] ¦ rakudo: [22:19] ¦ rakudo: - move the main logic to the Str class [22:19] ¦ rakudo: - make the sub version just pass on to the appropriate method [22:19] ¦ rakudo: - makes uninames about 2x as fast [22:19] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/d14dda68da