http://mbostock.github.com/d3/ex/bullet.html Displays no chart on ToT, but it works on Safari 5.1.2.
<rdar://problem/10650878>
Created attachment 121766 [details] Patch
Comment on attachment 121766 [details] Patch Yay! r=me
Any performance impact?
> Any performance impact? Bencher says 1.002x worse on SunSpider and v8 -- seems reasonable to call that "no change". Bytecode generation for f.apply is definitely worse by one op_mov. This is theoretically fixable with more robust optimization information in the AST, but I think that's probably the wrong direction -- long-term, we just want to compile f.apply in the DFG, which will elide the op_mov.
(In reply to comment #5) > > Any performance impact? > > Bencher says 1.002x worse on SunSpider and v8 -- seems reasonable to call that "no change". > > Bytecode generation for f.apply is definitely worse by one op_mov. This is theoretically fixable with more robust optimization information in the AST, but I think that's probably the wrong direction -- long-term, we just want to compile f.apply in the DFG, which will elide the op_mov. Agree!
Committed r104762: <http://trac.webkit.org/changeset/104762>