freenode/#clasp - IRC Chatlog
Search
18:39:13
yonkunas
selwyn: Hi, how would you like to proceed with quickclasp? Currently someone needs to su to your account to do updates.
19:36:15
karlosz
Bike, drmeister: bir contification is in. in particular https://github.com/clasp-developers/Eclector/blob/fa652c5d9750c4cbdc43082a3e07243bd2e265e4/code/reader/read.lisp#L76 this guy is gone and no longer unwinds
20:44:36
Bike
dentist went long, sorry. the test case is just the relevant parts of architecture.builder-protocol. sicne you got it down to this one call it's not much
22:00:53
karlosz
drmeister: Writing temporary bitcode file to: #P"/Users/karlosz/clasp/src/lisp/kernel/lsp/predlib.ll"
22:00:57
karlosz
Time real(4.958 secs) run(4.958 secs) consed(369343584 bytes) interps(7) unwinds(0)
22:09:31
karlosz
and here's the new flamegraph for compiling predlib now that there's no more unwinding during compilation: ocf.io/~karlos/bir-no-more-unwind.svg
22:24:09
karlosz
we should make sure we don't regress this though. note to make sure compiling predlib doesn't unwind in the future
22:29:54
drmeister
We could expose a way to count the number of unwinds that happen when a particular form is evaluated.
0:12:31
karlosz
yeah once we expose a way to count the number of unwinds for a form evaluation we can just pop that into the regression tests
0:13:25
karlosz
Bike: it may be worht trying to make it more deterministic by implementing sets as lists instead
2:32:52
karlosz
and also it seemingly can't even reconstruct '(b (a . c)) from a cst with raw (a . c)
2:36:39
karlosz
nope, well scymtym added the test at the same time as the fix that was supposed to make the test work
2:38:06
karlosz
anyway i was going to try to make reconstruct faster since a lot of time is spent there