Review tracker
Created attachment 29490 [details] Call caching Implement basic infrastructure for vm reentry caching, use Array.prototype.filter as the test implementation.
Comment on attachment 29490 [details] Call caching There is extra whitespace on line 186 of Interpreter.h. You use i++ in a few places where ++i would better match our style. The CallFrameClosure struct has all of its members public but all of them have the m_ prefix in front of them. It seems to be the style elsewhere in JSC to remove the prefix if the class is actually a pure struct. In this case there are also two methods that operate on the members in nontrivial ways, but there are unlikely to be more. I suggest removing the m_ prefices and leaving the rest as-is. If this is unreasonable or bad, let me know. > void setArgument(int n, JSValuePtr v) I think that we could come up with better names for these parameters. I think v could be 'value', something similar for 'n'. Other than that, r=me.
This was landed in r42537.