02:22 sortiz joined
.oO(I love the smell of refactor in the morning)
08:09 RabidGravy joined
jnthn :) 09:00
jnthn slept non-awfully and might manage some refactoring now :)
dalek p: b466eea | tomboy64++ | / (4 files):
enable custom jars to be used for building the JVM backend
p: 10b44b7 | jnthn++ | / (4 files):
Merge pull request #281 from tomboy-64/tomboy64-gentoo

Enable usage of custom asm.jar, asm-tree.jar, jline.jar and jna.jar
RabidGravy harr 09:18
tomboy64 hails jnthn :) merci bien 09:31
jnthn Want to give the MoarVM one a try out on Windows (or maybe you already did that?), but I'm good with merging it otherwise too. 09:32
tomboy64 sorry, don't have a win environment available to do that 09:33
what about the (minimal) rakudo patch? 09:34
i noticed quite a backlog in pull requests there
jnthn I've got a win environment, can do it later :) 09:37
tomboy64 thanks 09:38
jnthn github.com/rakudo/rakudo/pull/758/...f9cf78ca65 looks OK in what it does, though does something weird with whitespace... 09:39
tomboy64 lol yeah 09:40
i only just noticed
jnthn :)
tomboy64 i have my tabs set to two chars indents
and your indentation is extra-weird - 1 tab + 4 chars 09:41
jnthn wow :)
I'm quite sure that's acciental. 09:42
tomboy64 i can correct that as well. which indentation then? 09:43
tabs? or a number of whitespaces? 09:44
jnthn Well, its a Makefile, so the non-continuation lines have to be tabs, so probably tabs is the least crazy :) 09:45
10:42 brrt joined
dalek kudo/nom: 0035cd7 | lizmat++ | src/core/Cursor.pm:
Be smarter about looping in MATCH
kudo/nom: 1e5955c | lizmat++ | src/core/Str.pm:
Be smarter about looping in .split/.trans
kudo/nom: 9b318e3 | lizmat++ | src/core/ (10 files):
Replace i = i + 1 by ++i

There is no performance reason anymore to use the more verbose + 1
lizmat jnthn: running with HARNESS_TYPE=6, it looks like the 198th test file always has "no subtests run" 11:38
and between 10 and 19 files later, it looks like it is just not starting another test file 11:39
the base process just hangs, probably waiting for a promise to be kept, and it is never kept
0% CPU
jnthn Hm, what emits no subests run? The harness? 11:49
lizmat TAP.pm6 11:50
jnthn Hm, so something going wrong with the it spawning or getting the output of that test I guess 11:51
198 is kinda close to 200
lizmat yeah
figured it's basically the 200 issue in disguise 11:52
jnthn Don't suppose you tried with MVM_SPESH_OSR_DISABLE, since the OSR threshold is 200?
lizmat ah, will try 11:53
jnthn: no difference with MVM_SPESH_OSR_DISABLE=1 11:59
is that the right name ?
yeah, looks like 12:00
so, no, that's not it, by the looks of it
dyncall has a DC_CALL_SYS_DEFAULT at 200 12:01
jnthn moar --help to check :) 12:02
I don't think NativeCall is being used anywhere by it though?
And that doesn't seem likely even so
lizmat just grepping for 200 :-)
jnthn Could always try the wider MVM_SPESH_DISABLE just to check :)
lizmat nope, the 198th test file fails, but it does seem to get further on from there 12:08
25 additional test files
so, feels to me there's multiple interaction bugs :-( 12:09
jnthn Well, it's possible that removing spesh simply changed some timings and made the smae thing come up later
lizmat ok, stopping this line of investigation until moar fixes land :-) 12:20
dalek kudo/nom: 810788b | lizmat++ | src/core/Any.pm:
Postfixize another for loop
lizmat afk& 13:19
13:20 skids joined
dalek kudo/moar/reframe: 2cbea3b | jnthn++ | src/vm/moar/ops/perl6_ops.c:
Chase MoarVM reframe branch changes.
p: 04a10f7 | (Pawel Murias)++ | src/vm/js/nqp-runtime/runtime.js:
[js] Export loadOps to enable loading the rakudo specific ops.
p: 0d4b885 | (Pawel Murias)++ | src/vm/js/DWIMYNameMangling.nqp:
[js] More pleasant to read name mangling.
p: ea94370 | (Pawel Murias)++ | src/vm/js/Operations.nqp:
[js] Implement nqp::takedispatcher/nqp::cleardispatcher/nqp::setdispatcher.
tomboy64 jnthn: do you have any idea how to get the build log file by appveyor? 18:25
jnthn tomboy64: No...haven't had chance to look at appveyor at all yet 18:26
tomboy64 meh. 18:28
alright, so no indentation improvements for rakudo :p
19:14 brrt joined
[Coke] tomboy64: do you mean .indent, or source code indentation? 19:48
huh. I see tabs have snuck in. evil.
[Coke] cleans up src/ 19:51
dalek kudo/nom: 8ef120c | coke++ | src/ (4 files):
replace tabs with spaces in source
kudo/nom: b05593c | coke++ | / (4 files):
replace tabs with spaces in source
tomboy64 [Coke]: bpaste.net/show/8e5b5c93f00e <--- this makes stuff fail with appveyor checks and i'm not in the mood to check myself :p 20:04
[Coke]: github.com/rakudo/rakudo/pull/758 this is the pr it would apply against. it won't apply against stock tree. 20:08
[Coke] are you just shortening lines in that bpaste? 20:21
makefiles can have spaces and tabs, and tabs mean something very specific. the commit message on a56911e doesn't give the feeling that was taken into account.
there is no need to have line 65's leading space be a tab, for example. 20:22
(which it looks like that commit changes) 20:23
So I probably would not merge 758 as it stands. 20:24
0caa945f00 seems reasonable to apply. 20:25
tomboy64 oh
RabidGravy yeah for almost all makes the command lines have to be prefixed by a tab
tomboy64 shrugs
i edited that because in the original files there are leading tabs/spaces mixed
[Coke] right, but that commit is changing things that are continuation lines, which aren't commands. Having tabs there is not technically wrong, but more stylistically wrong.
tomboy64: yes.. 20:26
welcome to makefiles.
tomboy64 yay
tomboy64 reverts a569
[Coke]: okay, removed the tab-"fix" 20:34
[Coke]: also, i'm experiencing build failures with the jvm backend whenever i try to build with rakudo already being installed. 20:35
would you be interested in helping me debug that? 20:36
dalek kudo/nom: 780f073 | lizmat++ | src/core/ (2 files):
Make LoL accesses about 3x as fast

  - by introducing a new WALK-AT-POS method in Rakudo::Internals
  - using that method in Any.(DELETE|EXISTS|AT|ASSIGN)-POS
  - alas, BIND-POS did not work correctly with this treatment
   it looks like Array.pop is somehow returning a different container
   than an AT-POS does :-(
timotimo wow, 3x, that's fantastic 21:59
probably still rather expensive?
21:59 Skarsnik joined
lizmat well, the AT-POS itself is still rather expensive 21:59
but walking the indices array is now 3x as fast
timotimo oh, i see 22:00
lizmat m: my @a = [1,2],[3,4]; dd @a.AT-POS(0,0) 22:01
camelia rakudo-moar b05593: OUTPUT«Int <element> = 1␤»
lizmat that part has gotten faster
m: my @a = [1,2],[3,4]; dd @a[0;0] # how you would write this normally
camelia rakudo-moar b05593: OUTPUT«Int <element> = 1␤»
timotimo oh, ah
lizmat not sure how fast it goes from [0;0] to AT-POS(0,0) 22:02
timotimo mhm
lizmat looks like direct AT-POS is 2x as fast as [0;0] 22:03
room for improvement there
timotimo wow 22:04
lizmat but first, sleep&
timotimo gnite!