LGTM
http://codereview.chromium.org/1611021/diff/1/2 File src/array.js (right): http://codereview.chromium.org/1611021/diff/1/2#newcode715 src/array.js:715: var key = %_IsSmi(element) ? element : ToString(element); I'm very much attached to this line, but on the other hand, the most common uses of sort-with-default-comparison is on numbers and strings, so it might just be overhead. Keep it, but maybe try a test run later without this line and see if it makes a difference in any direction. http://codereview.chromium.org/1611021/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev To unsubscribe, reply using "remove me" as the subject.
