Make destructuring emit direct keyword lookups

Description

Currently associative destructuring emits calls to get. The attached patch modify desctruture to emit direct keyword lookups when possible.

Approved here https://groups.google.com/d/msg/clojure-dev/MaYcHQck8VA/nauMus4mzPgJ

Environment

None

Activity

Show:
Christophe Grand
September 4, 2013, 9: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.

Christophe Grand
September 4, 2013, 9:41 AM

More generic patch (inline-get-keyword.diff): all get calls with literal keywords as keys are inlined to direct keyword lookup.

John Hume
May 19, 2014, 7:14 PM

Is this only stalled out of lack of interest?

Andy Fingerhut
May 20, 2014, 12:13 AM

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.

Assignee

Christophe Grand

Reporter

Christophe Grand

Approval

Triaged

Patch

Code

Affects versions

Priority

Major