github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
brrt \o 09:46
nwc10 o/ 09:53
brrt I have a ... challenge? problem? design question... 10:00
The template language specifies a branching operation as a tree of branch -> test -> operands 10:01
(or: operands, test, branch sequence)
however, in emitting the branch opcode, we kind of need to know whether the opcodes where floating points, or integers 10:02
since x86 sets every so slightly different combinations of flags for floating point and integer register comparison 10:03
(it doesn't matter whether you use SSE or x87 FPU for the oprations, fwiw, although the x87 FPU has special 'internal' branching opcodes as well)
The design question is to have a nice way, that doesn't break, to transfer the type of the operands 'up' to the branch 10:05
nwc10 I don't know enough about the current code to be able to answer that. Or even where to start answering it. Acutally 10:17
is this the only situation where the parent or grandparent "op" needs to know something from one of its children? 10:18
timotimo well, i can't quite believe i actually made this work 13:38
now to make it work maybe fast
timotimo 736.15user 3.87system 4:51.73elapsed 253%CPU (0avgtext+0avgdata 1190144maxresident)k 14:51
well, that's surely about 2.5x faster than 1x speed 14:52
lizmat timotimo: and what is now faster ? 14:57
timotimo oh, you know, just some trash 14:58
this code will have to be torn to pieces and reassembled into something more usable before it can go into the Moar Heapanalyzer
it's for reading the heap snapshot format that moar will spit out in the future
lizmat aaahhhh more and faster introspection ! 14:59
timotimo well, maybe 15:00
at least the files are now much smaller
er, at least i think they are 15:01
brrt nwc10: Not exactly, there's also operand size 18:09
that's not exactly relevant to flag operators, so..... 18:11
I smell the potential for reusing a field
(you're not *really* a C programmer until you've used a field in multiple ways depending on implicit context) 18:21
TimToady hides in pride^Wshame 18:38
Geth MoarVM: patzim++ created pull request #1131:
Build with spaces in path		 21:58
patrickb ^- Not for merge before the release! 22:10
AlexDaniel samcv: speaking of which, how is it going? :) 22:11
MasterDuke didn't github recently announce the concept of 'draft PRs'?
Kaiepi yes 22:43
i've been using them
patrickb I found the respective button. Good to know for next time! 22:48
Kaiepi they're great for wip pullreqs 23:26

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!