[CLJ-1096] Make destructuring emit direct keyword lookups Created: 29/Oct/12 Updated: 06/Jun/14
|Affects Version/s:||Release 1.4, Release 1.5, Release 1.6|
|Reporter:||Christophe Grand||Assignee:||Christophe Grand|
Currently associative destructuring emits calls to get. The attached patch modify desctruture to emit direct keyword lookups when possible.
|Comment by Christophe Grand [ 04/Sep/13 3:40 AM ]|
Rethinking about this patch now, it may be too specific: get's inline expansion should be modified when the key is a literal keyword.
|Comment by Christophe Grand [ 04/Sep/13 3:41 AM ]|
More generic patch (inline-get-keyword.diff): all get calls with literal keywords as keys are inlined to direct keyword lookup.
|Comment by John Hume [ 19/May/14 1:14 PM ]|
Is this only stalled out of lack of interest?
|Comment by Andy Fingerhut [ 19/May/14 6:13 PM ]|
There are currently about 50 tickets "triaged", i.e. marked for Rich to look at and decide whether they are things he is interested in seeing a patch for, and another 25 or so that were triaged and he has "vetted" them, and they are in various stages of having patches written for them, screened, etc. That doesn't mean anything for this ticket in particular – just wanted to make it clear that there are a bunch of other tickets that are getting some attention, and a bunch of others that are not.
What gets triaged depends somewhat upon how severe the issue appears. You can vote on the ticket, and try to persuade others to do so as well, if they think this would enhance the performance of some commonly-written types of Clojure code. You could also consider doing some benchmarking with & without these patches to see how much performance they can gain.