freenode/#clasp - IRC Chatlog
Search
13:16:46
Bike
drmeister: i deleted simplebitvector copy because i didn't see anything using it, but now i see it's in cando. i thought i checked, sorry
13:16:46
Colleen
Bike: drmeister said 13 minutes, 58 seconds ago: I'm trying to listen to warnings more. It appears that if we have a (defgeneric foo ...) followed by a (defmethod foo ...) the (defmethod foo ...) generates a style warning Undefined function. Or am I misinterpreting something?
13:17:09
Bike
drmeister: yeah defgeneric doesn't register with the compiler, i noticed that too. shouldn't be hard to fix
13:17:47
drmeister
::notify kpoeck I was mistaken when I thought that you deleted SimpleBitVector_copy - sorry about that. :-)
13:18:43
Shinmera
drmeister: Bike: In case you missed it yesterday, in order to add support for Clasp to Dissect, I'd be very grateful if you could fill out this survey. https://plaster.tymoon.eu/view/856#856 (click the annotate button at the bottom)
13:21:37
Shinmera
Yeah I understand it interleaves C frames, but I can deal with that as long as the survey is answered :)
13:36:40
drmeister
Bike: It's becoming crystal clear to me that cclasp's slow compiler performance is due to the slow performance of the llvm back-end - and I don't see a lot of fat that can be trimmed there.
13:45:44
Bike
and by the way,if it wasn't clear, there's a lot more warnings during build now because i found we were improperly suppressing a lot of them
13:46:07
Bike
now there are a lot from the LOADs, because during load undefined functions notes aren't delayed like they are during compile-file
14:04:33
Bike
the thing looks for universal-error-handler, but for whatever reason the backtrace has the string "UNIVERSAL-ERROR-HANDLER" instead of the symbol
14:05:32
drmeister
Right - C frame names are strings, Common Lisp function names are symbols. What do I do?
14:45:46
Colleen
kpoeck: drmeister said 2 hours, 30 minutes ago: You removed SimpleBitVector_copy - what was the reason? I ask because cando used it and it has a non-trivial length calculation that factors in the size of the bit units. I appreciate cleanup very much - but I restored this function for now.
14:45:46
Colleen
kpoeck: drmeister said 2 hours, 28 minutes ago: If SimpleBitVector_copy is redundant in some way I'm happy to remove it - I just don't recall if it is.
14:45:46
Colleen
kpoeck: drmeister said 1 hour, 27 minutes ago: I was mistaken when I thought that you deleted SimpleBitVector_copy - sorry about that. :-)
14:52:30
beach
What do people here think of the idea of introducing HIR "instructions" to mark the beginning and the end of the execution of a form?
14:55:10
heisig
Is that an improvement over having the precise source location attached to each instruction? And what would be the rules for reordering these instructions during optimization?
14:56:12
beach
The reason I am thinking about it is that I want support for the step-over command in the debugger.
14:56:28
beach
So, given the beginning of a form, it must find the PC value corresponding to the end of the form.
15:00:03
heisig
Since the debugger is not performance-critical, can it not just search all instructions and choose the one with the most appropriate source location?
15:02:58
heisig
I am not sure. But I think one could develop a robust heuristic for what is "appropriate".
15:22:41
nivpgir
Hi, so apparently building on my machine still fails, any special dependencies on arch linux I should know of?
15:23:48
nivpgir
And if not, can someone help me find the error, I'm not exactly proficient with waf
16:01:56
kpoeck
@nivpgir: I must have built clasp more times that had birthdays. If the problem is easy, I might be able to help, if not easy than you would need the real experts
16:26:53
nivpgir
Kpoeck: thanks, I'll be in front of my pc soon and I'll send here the error message?
16:27:44
Colleen
Unknown command. Possible matches: 8, set, say, mop, roll, get, search, login, award, time,
17:22:48
jackdaniel
this may be affecting clasp too: https://gitlab.com/embeddable-common-lisp/ecl/issues/445
17:29:51
kpoeck
@jackdaniel, will test your example. Also found some equalp errors in clasp, but that code in clasp is not easy
17:30:40
Bike
yeah, we kind of have our own definitions for a lot of that stuff, but thanks for the heads up
17:32:38
jackdaniel
sure. it happens to be in ECL's C code and I know that Clasp core was more-or-less rewritten to C++ verbatim algorithm-wise
19:50:34
karlosz
so partial inlining generated a graph where an eq instructions two successors are the same
19:55:02
karlosz
okay, then i need to fix the basic blocks code to not use pushnew when calculating preds and successors
19:55:41
karlosz
it would be nice to eliminate those eq instructions though, i think they can just be deleted
20:02:47
karlosz
maybe it can be done somewhere in inlining, since thats how they are introduced, i think
20:23:46
karlosz
thats true. it seems like an easy invariant to enforce, and it probably doesnt warrant an entire separate pass like delete-superfluous-branches or something