dalek | p: ea1104f | MasterDuke17++ | src/QRegex/Cursor.nqp: Remove only use of an internal opcode in the src |
00:54 | |
babydrop | What's wrong with nqp::ordat()? | 01:00 | |
In irclog.perlgeek.de/perl6/2016-11-29#i_13646879 timotimo says that ordat, as opposed to ord, would "circumvent an allocation that'll immediately get cleaned up again, but it goes through malloc, too" | 01:01 | ||
MasterDuke | hm, the nqp doc said it was an internal compiler opcode, and that was the single use in all the source code | 01:09 | |
babydrop | Cool | 01:10 | |
MasterDuke | is ordat(str, 0) better than ordfirst(str)? that's the other internal opcode which seems like a better fit | ||
babydrop has no idea | |||
dalek | kudo/nom: 0ba7733 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Grammar.nqp: Fix RT #130260 (“smart” → “curly”) This seems to be the only place where these quotes are called this way. |
01:11 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130260 | ||
dalek | kudo/nom: 40d80c0 | lizmat++ | src/Perl6/Grammar.nqp: Merge pull request #935 from AlexDaniel/curly-quotes Fix RT #130260 (“smart” → “curly”) |
||
babydrop | the docs say "ord is converted to these internal opcodes by the compiler.", so I guess it just chooses which to convert to depending on the index given | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130260 | ||
MasterDuke | if ordat really is faster, there are a bunch of ord()s in MATCH() that could be changed (and it might make sense there since it is such a hot path) | 01:12 | |
babydrop | I probably just misunderstood timotimo, since I was talking about doing nqp::ord(nqp::substr(...)) | 01:14 | |
and that's not needed, 'cause ord takes an index (and I didn't know) | 01:15 | ||
MasterDuke | .ask timotimo if you backlog you'll see a discussion about nqp::ord() and nqp::ordat(). is nqp::ordat(str, index) really better than nqp::ord(str, index)? | 01:17 | |
yoleaux2 | MasterDuke: I'll pass your message to timotimo. | ||
dalek | p: 4bded8d | MasterDuke17++ | src/vm/moar/HLL/Backend.nqp: Fix typo in SQL profile output column name |
01:23 | |
ugexe | bisect: {class Test::Builder::Test { };}; {role Test::Builder::Plan::Generic { }; class Test::Builder::Plan does Test::Builder::Plan::Generic { };}; {class Test::Builder { has Test::Builder::Plan::Generic $!plan; };}; | 03:29 | |
bisectable6 | ugexe, Bisecting by exit code (old=2015.12 new=40d80c0). Old exit code: 0 | ||
ugexe, bisect log: gist.github.com/439c70938529dc4a09...9e31b88f45 | |||
ugexe, (2016-10-26) github.com/rakudo/rakudo/commit/ab...8acd5bfc71 | |||
ugexe | Seems to be related to the first name part (Test) matching the last name part (Test), because if you change the first name part to 'Tester' it works again | 03:35 | |
m: {class Tester::Builder::Test { };}; {role Tester::Builder::Plan::Generic { }; class Tester::Builder::Plan does Tester::Builder::Plan::Generic { };}; {class Tester::Builder { has Tester::Builder::Plan::Generic $!plan; };}; | 03:36 | ||
camelia | ( no output ) | ||
ugexe | m: {class A::B::A { };}; role A::B::C { }; class A::B { has A::B::C $!plan; }; | 03:50 | |
camelia | rakudo-moar 40d80c: OUTPUT«===SORRY!===Type 'A::B::C' is not declaredat <tmp>:1------> le A::B::C { }; class A::B { has A::B::C⏏ $!plan; };Malformed hasat <tmp>:1------> ole A::B::C { }; class A::B { has A::B::⏏C $!plan; };» | ||
ugexe | m: class A::B::A { }; role A::B::C { }; class A::B { has A::B::C $!plan; }; | 03:51 | |
camelia | ( no output ) | ||
ugexe | for some reason wrapper that first class in { }; causes it there | ||
unless you change A::B::A to A::B::<any thing but A> | 04:04 | ||
m: {class A::B::X { };}; role A::B::C { }; class A::B { has A::B::C $!plan; }; | |||
camelia | ( no output ) | ||
dalek | kudo/nom: 6fafbf6 | niner++ | tools/build/create-moar-runner.pl: Hopefully fix make test on Windows The perl6-m.bat in the build directory was missing the --nqp-lib=blib argument. |
07:01 | |
nine | Could also be that we need an NQP bump | 07:03 | |
[TuxCM] | This is Rakudo version 2016.11-132-g40d80c02a built on MoarVM version 2016.11-27-g3171dbbe | 07:12 | |
csv-ip5xs 3.061 | |||
test 15.116 | |||
test-t 6.658 | |||
csv-parser 14.830 | |||
bartolin | for the records: the jvm backend is completely broken atm. yesterday it failed after Stage optimize with 'classfile too large' (or similar) and today it fails even earlier with 'NoSuchFileException: /share/nqp/lib/Perl6/BOOTSTRAP.jar' | 07:15 | |
bartolin is off to $work, so won't have time to look until later | |||
(the above is for building rakudo, nqp-j builds fine) | 07:16 | ||
nine | Ok, Appveyor is happy again :) | 07:32 | |
bartolin: the BOOTSTRAP.jar issue is most probably my fault :/ I only changed the Makefile-Moar.in. Train ride was over before I could attack Makefile-JVM.in | 07:33 | ||
It just needs those --nqp-lib=blib arguments | 07:34 | ||
lizmat | Files=1155, Tests=53838, 170 wallclock secs ( 9.44 usr 4.03 sys + 976.13 cusr 110.62 csys = 1100.22 CPU) | 08:01 | |
finally got it to run on my new MBP when it was cold | 08:02 | ||
nine | So the new hardware is about 1/4 faster | ||
lizmat | more like 20% | 08:05 | |
running a jvm build while going for breakfast& | |||
Stage parse : java.nio.file.NoSuchFileException: /share/nqp/lib/Perl6/BOOTSTRAP.jar | 08:56 | ||
I guess that's what nine was referring to | 08:57 | ||
decommute& | |||
dalek | kudo/nom: b96bf5b | niner++ | tools/build/Makefile-JVM.in: Fix java.nio.file.NoSuchFileException: /share/nqp/lib/Perl6/BOOTSTRAP.jar We no longer search . and blib by default when looking for NQP modules. |
09:57 | |
timotimo | MasterDuke: yeah, ordat is only better than ord + substr | 10:17 | |
yoleaux2 | 01:17Z <MasterDuke> timotimo: if you backlog you'll see a discussion about nqp::ord() and nqp::ordat(). is nqp::ordat(str, index) really better than nqp::ord(str, index)? | ||
dogbert17 | nine: are you around? | 10:46 | |
babydrop | [TuxCM]: are those times affected by precompilation or do you precompile first and then run the bench? | 11:36 | |
I noticed it got .2s slower after the day when I made − work in many places. | |||
And it was changes in P6/Grammar. | |||
jnthn | .2s is probably in noise | 11:37 | |
babydrop | seems a pretty consistent noise :D | ||
jnthn | I guess the same thing over a few days makes it more significant, true :) | 11:38 | |
nine | dogbert17: yes | 11:57 | |
dogbert17 | nine: would it be possible for you to take a look at the 32 bit invalid read errors that still plague t/04-nativecall/08-callbacks.t? | 12:12 | |
nine: time permitting ofc | 12:14 | ||
jnthn | dogbert17: May also be worth mentioning that to FROGGS++ if you didn't already; he's done quite some NativeCall platform fixes recently :) | 12:19 | |
nine | dogbert17: yes, if FROGGS++ doesn't beat me to it, I can have a look tomorrow. | ||
dogbert17 | nine froggs: here's what I have: gist.github.com/dogbert17/c88ee798...e4960e885b | 12:20 | |
nine: have updated the gist, I had forgotten to turn on debug info /o\ | 12:25 | ||
dalek | kudo/nom: 6e7f97f | (Zoffix Znet)++ | src/Perl6/Grammar.nqp: Microoptimize synth rejector code in colonpairs Gives about a 13% boost + likely memory improvements |
14:41 | |
[Coke] | bdf reports: | 14:49 | |
I just noticed in another editor that the Perl 6 Advent wordpress | |||
setup replaces HTML escapes when it populates the edit window. Things | |||
that were < become literal < again. Odd. | |||
babydrop | :o | ||
gfldex's post still has an issue.. it should trait_mod:<is> or something | 14:50 | ||
[Coke] | -- | 14:51 | |
... wrong window. whoops. | |||
babydrop | ... | 14:56 | |
ZzZombo's quit messages get progressively more bizzare :) | 14:57 | ||
gfldex | babydrop: fixed | 15:28 | |
babydrop | awesome | ||
[Coke] | gfldex++ | ||
babydrop | .tell Zoffix remember the for <a b> \ <b c> { .say } bug? Did you fix it? | 17:49 | |
yoleaux2 | babydrop: I'll pass your message to Zoffix. | ||
babydrop | #LazyBugReporting | ||
.tell Zoffix never mind | 17:53 | ||
yoleaux2 | babydrop: I'll pass your message to Zoffix. | ||
geekosaur | heh | 17:59 | |
japhb | .oO( Zoffix will eventually implement Eiffel-on-wetware to get reliable forward progress despite constantly crashing personality nodes ) |
18:07 | |
dalek | p: 671b680 | (Pawel Murias)++ | src/vm/js/Chunk.nqp: [js] Remove no longer used ChunkEscaped class. |
18:48 | |
p: 6e62355 | (Pawel Murias)++ | src/vm/js/ (2 files): [js] Return the exit status from nqp::spawn and nqp::shell. |
|||
p: 7695020 | (Pawel Murias)++ | t/nqp/078-shell.t: Test the return value of nqp::shell. |
|||
p: 642f1e5 | (Pawel Murias)++ | src/vm/js/ (2 files): [js] Avoid duplicating a block by setting it up with both a .closure and .capture. |
|||
kudo/remove-useless-check: ab10dc4 | (Pawel Murias)++ | src/Perl6/Actions.nqp: Don't check for an op that doesn't exist. |
20:28 | ||
japhb | pmurias++ # Relentless(ly wonderful) expansion of the nqp test suite | 20:46 | |
psch | m: class A {}; say ::('::x::A') | 20:48 | |
camelia | rakudo-moar 6e7f97: OUTPUT«No such symbol 'A' in block <unit> at <tmp> line 1Actually thrown at: in block <unit> at <tmp> line 1» | ||
psch | m: class A {}; say ::('::A') | ||
camelia | rakudo-moar 6e7f97: OUTPUT«(A)» | ||
psch | oh, hm, saw that in #perl | 20:49 | |
+6 | |||
>_> | |||
bartolin | ah, my memory didn't fool me: we had that 'class file too large' problem with rakudo-j earlier: irclog.perlgeek.de/perl6/2015-10-27#i_11439297 | 21:04 | |
timotimo | yup, we did | 21:05 | |
we did a trick to make it better | |||
but it was only a measure to push the problem back | |||
bartolin | yeah, nqp commit 61f3e95ec4 by psch++ | 21:06 | |
[Coke] | pmurias++ ugh and it was the first check, too! | 21:10 | |
timotimo | i expect many of these eq checks were mostly pointer comparison shortcuts | 21:11 | |
because almost all values put into :op come directly from CompUnit's string heap | |||
and thus ought to be interned | |||
probably not across CompUnits, though | |||
dalek | kudo/nom: ab10dc4 | (Pawel Murias)++ | src/Perl6/Actions.nqp: Don't check for an op that doesn't exist. |
21:33 | |
kudo/nom: 7ad4a86 | jnthn++ | src/Perl6/Actions.nqp: Merge pull request #937 from rakudo/remove-useless-check Don't check for an op that doesn't exist. |
|||
jnthn | super tired; getting an early night & | 21:36 | |
timotimo | me, too | ||
lizmat | gfldex: re "please add to the next weekly that the commits 19169cc, 3ffdc37, b9a6dc8, 0bf8b18, c7b14ae, ce7c9f6 clarify the relationship between containers, default types and type constraints -- what used to be a rich source of confusion", not sure how to put that in a P6W entry in a nice readable, clickable form :-( | 22:39 | |
japhb | lizmat: Do those commits all point to the same doc or two? You could point to those as being rewritten for clarity .... | 22:44 | |
babydrop | lizmat: "There were a <a href="github.com/perl6/doc/compare/19169...>number of improvements to documentation,</a> including clarification to containers, types, and type constraints | 22:45 | |
well, github.com/perl6/doc/compare/19169cc...e01fd79 since master will move | 22:46 | ||
japhb | .ask jnthn You mentioned that concurrency was going to remain on your brain for at least the first half of 2017. Do you have any idea how much remains in the "make it work reliably" category, and when you'll be able to work on "make it fast/scalable/efficient"? I'm starting to write code that seems to be hitting non-linearities in the scheduler. | 22:47 | |
yoleaux2 | japhb: I'll pass your message to jnthn. | ||
japhb | .ask jnthn Also, is it possible to reserve a single real thread for the main app thread? I've got some high-concurrency code where the main thread (which actually does the I/O in this case) gets very janky, and I'm guessing that user-visible task is getting starved for CPU resources by lots of background tasks. | 22:49 | |
yoleaux2 | japhb: I'll pass your message to jnthn. | ||
lizmat | and another issue of Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2016/12/05/...log-posts/ | 22:52 | |
tadzik | \o/ | 23:04 |