LGTM. You can have an mjsunit test that allows native syntax by adding a comment line of the form: // Flags: --allow-natives-syntax in the file. Check, e.g., array-sort.js
http://codereview.chromium.org/437081/diff/1/4 File src/compiler.cc (left): http://codereview.chromium.org/437081/diff/1/4#oldcode983 src/compiler.cc:983: if (expr->function() == NULL) BAILOUT("call JS runtime function"); On general principle, could we put an is_js_runtime_function method on CallRuntime that tested the function against NULL, instead of having to know the significance of a NULL function everywhere? The name probably needs work. http://codereview.chromium.org/437081 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
