[00:33] ¦ MoarVM/release/2018.04.1: a08c4fd548 | (Samantha McVey)++ | 2 files [00:33] ¦ MoarVM/release/2018.04.1: Enable TravisCI and Appveyor for 2018.04.1 release branch [00:33] ¦ MoarVM/release/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/a08c4fd548 [00:43] *** travis-ci joined [00:43] MoarVM build failed. Samantha McVey 'Enable TravisCI and Appveyor for 2018.04.1 release branch' [00:43] https://travis-ci.org/MoarVM/MoarVM/builds/372839860 https://github.com/MoarVM/MoarVM/compare/43f9dcfe83fb...a08c4fd54842 [00:43] *** travis-ci left [00:45] ¦ MoarVM/release/2018.04.1: ff5f9f1f38 | (Samantha McVey)++ | 2 files [00:45] ¦ MoarVM/release/2018.04.1: Test 2018.04 version of nqp on CI [00:45] ¦ MoarVM/release/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/ff5f9f1f38 [00:51] *** travis-ci joined [00:51] MoarVM build errored. Samantha McVey 'Test 2018.04 version of nqp on CI' [00:51] https://travis-ci.org/MoarVM/MoarVM/builds/372841632 https://github.com/MoarVM/MoarVM/compare/a08c4fd54842...ff5f9f1f38b1 [00:51] *** travis-ci left [00:53] ¦ MoarVM/release/2018.04.1: cf4a108d78 | (Samantha McVey)++ | 2 files [00:53] ¦ MoarVM/release/2018.04.1: Test 2018.04 version of nqp on CI [00:53] ¦ MoarVM/release/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/cf4a108d78 [01:00] *** Kaiepi joined [01:09] *** travis-ci joined [01:09] MoarVM build passed. Samantha McVey 'Test 2018.04 version of nqp on CI' [01:09] https://travis-ci.org/MoarVM/MoarVM/builds/372842934 https://github.com/MoarVM/MoarVM/compare/ff5f9f1f38b1...cf4a108d785b [01:09] *** travis-ci left [01:11] ¦ MoarVM/release/2018.04.1: 35e6693411 | (Samantha McVey)++ | docs/ChangeLog [01:11] ¦ MoarVM/release/2018.04.1: Update ChangeLog for release [01:11] ¦ MoarVM/release/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/35e6693411 [01:16] ¦ MoarVM/refs/tags/2018.04.1: e00dc3e53b | (Samantha McVey)++ | VERSION [01:16] ¦ MoarVM/refs/tags/2018.04.1: Release 2018.04.1 [01:16] ¦ MoarVM/refs/tags/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/e00dc3e53b [01:38] *** FROGGS__ joined [01:56] *** ilbot3 joined [01:56] *** moderator changes topic to: https://github.com/moarvm/moarvm | IRC logs at http://irclog.perlgeek.de/moarvm/today [03:42] ¦ MoarVM/master: 4 commits pushed by (Zoffix Znet)++, (Samantha McVey)++, (Aleks-Daniel Jakimenko-Aleksejev)++ [03:42] ¦ MoarVM/master: 5d36d30a0b | Fix denormals and precision issues in num parser [03:42] ¦ MoarVM/master: 43f9dcfe83 | Remove trailing whitespace [03:42] ¦ MoarVM/master: 35e6693411 | Update ChangeLog for release [03:42] ¦ MoarVM/master: f583e25c9c | Merge branch 'release/2018.04.1' [03:42] ¦ MoarVM/master: review: https://github.com/MoarVM/MoarVM/compare/c702f4c66aa8...f583e25c9c67 [04:28] *** AlexDaniel joined [05:06] ¦ MoarVM/release/2018.04.1: e00dc3e53b | (Samantha McVey)++ | VERSION [05:06] ¦ MoarVM/release/2018.04.1: Release 2018.04.1 [05:06] ¦ MoarVM/release/2018.04.1: review: https://github.com/MoarVM/MoarVM/commit/e00dc3e53b [05:08] ¦ MoarVM: e00dc3e53b | (Samantha McVey)++ | VERSION [05:08] ¦ MoarVM: Release 2018.04.1 [05:08] ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/commit/e00dc3e53b [05:08] ¦ MoarVM: af60a86106 | (Aleks-Daniel Jakimenko-Aleksejev)++ | VERSION [05:08] ¦ MoarVM: Merge branch 'release/2018.04.1' [05:08] ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/commit/af60a86106 [06:16] *** robertle joined [07:23] *** zakharyas joined [07:38] *** domidumont joined [07:44] *** domidumont joined [08:10] *** domidumont joined [08:18] *** zakharyas joined [10:01] *** domidumont joined [11:11] *** Ven`` joined [12:39] *** zakharyas joined [12:58] *** AlexDaniel joined [13:30] *** AlexDaniel joined [13:38] *** brrt joined [13:41] jnthn: thoughts on naming on this https://github.com/samcv/MoarVM/blob/siphash/src/strings/ops.c#L2893 [13:41] specifically MVM_MAYBE_TO_LITTLE_ENDIAN_32. description of what it does is in the linked comment [13:42] do we ever export the hash code? [13:42] do we make any guarantees whatsoever about it? [13:44] brrt: no we don't [13:46] also i'd like to switch to 64 bit hashes instead of 32 bit, not sure if it's just as easy as changing the data type in uthash.h [13:46] i recall it using &'s to decide which bucket to put things in [13:49] brrt: that MVM_MAYBE_TO_LITTLE_ENDIAN is also used in the hashing implementation file as well, just to ensure we can show that things aren't broken on different endianess. [13:53] why 64 bit hashes? [13:55] samcv: Why do we need the MVM_MAYBE_TO_LITTLE_ENDIAN_32 if we already guard it with #if defined(MVM_HASH_FORCE_LITTLE_ENDIAN? [13:55] i don't know how, but i can no longer install moar using the mingw toolchain coming with strawbetrry perl [13:56] jnthn: it's used many places not just for the hash secret [13:56] for the 32 bit codepoints, and also in the hashing code [13:56] ah, ok [13:56] I guess the name is OK [13:56] MVM_TO_LITTLE_ENDIAN_IF_FORCED is more explicit [13:57] ah that sounds good [13:57] Well, more explicitly links it with MVM_HASH_FORCE_LITTLE_ENDIAN at least :) [13:58] *** Ven`` joined [14:13] brrt: well siphash creates a 64 bit hash [14:30] that seems valid enough [14:56] *** zakharyas joined [15:47] *** Ven`` joined [15:50] *** robertle joined [16:07] hello wonderful #moarvm people! [16:07] Please take a look at this: https://irclog.perlgeek.de/perl6/2018-04-30#i_16114032 [16:08] that looks like a patch we could totally apply right now, but we may want to actually CAS the result at the end so we know to delete the stuff we just created if someone else beat us to it [16:09] I also wonder why this gets called twice, is that perhaps an indication of a bigger problem? [16:11] it's a benign race in case the bytecode only needs to be validated, not changed, and with your patch it's a benign race that just leaks a little bit of memory in case endian swapping is needed [16:44] *** Ven`` joined [17:31] *** zakharyas joined [18:05] *** Ven`` joined [18:33] *** Ven`` joined [19:40] *** evalable6 joined [20:15] *** brrt joined [20:40] . [20:47] yo brrt [20:48] would the spesh slot value/address types for emit_c_call be super easy to do? [20:55] yes [20:56] very [21:14] *** robertle joined [21:25] oh, i didn't look back for an answer [21:25] i'd appreciate if we had it :) [21:25] but really only for the three ops i recently added :D [21:30] *** brrt joined [21:43] and another Perl 6 Weekly hits the Net: https://p6weekly.wordpress.com/2018/04/30/2018-18-releases-galore/ [21:51] lizmat++ [21:52] *** jnthn joined [23:01] unless objections, i'm going to hit the merge button on the spesh-debug-limit branch [23:22] *** notable6 joined [23:22] *** bloatable6 joined [23:22] *** evalable6 joined [23:22] *** nativecallable6 joined [23:22] *** benchable6 joined [23:22] *** committable6 joined [23:22] *** greppable6 joined [23:22] *** coverable6 joined [23:22] *** quotable6 joined [23:22] *** releasable6 joined [23:22] *** reportable6 joined [23:22] *** squashable6 joined [23:22] *** bisectable6 joined [23:22] *** unicodable6 joined [23:22] *** statisfiable6 joined [23:41] *** brrt joined