freenode/#clasp - IRC Chatlog
Search
21:39:55
Bike
not gonna push until tomorrow though, cando might be weird since i deleted a bunch of C++ stuff
23:38:57
karlosz
its here. someone will have to decide to merge it https://github.com/robert-strandh/SICL/pull/130
23:50:39
Bike
mostly for some sicl stuff, but it is a nice little performance improvement for some things... :after methods are one
0:45:22
karlosz
next target is getting rid of reinitialize data in the inner loop of the inlining pass
0:46:07
Bike
yeah, that's tricky... but it only needs to be enough to support the closure flow analysis
0:56:28
karlosz
and i think i've found a compromise to this dreaded, recalculate everything vs incrementally update but break abstraction debate
0:57:11
karlosz
just provide incremental versions of each of the set-predecessor, remove-useless-instruction, reinitialize-data functions, so that each call site can use them without breaking abstraction but not recalculate info all the time too
1:00:01
Bike
how do you know which instructions to run this on, or which are potential predecessors?
1:05:50
karlosz
here is how i imagine this instruction will be used: everytime you want to delete an instruction x, you get the predecessors p of that instrution x, consider the successors s of x, you delete x, then you call this reinitialize function on p and s
1:06:17
karlosz
but the point is, there are things that make it so you need to call set-predecessors