Reviewers: Michael Starzinger,
Description:
Remove arguments arity check in mjsunit because it vastly slows down the
tests.
[email protected]
BUG=
Please review this at https://codereview.chromium.org/724133004/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+2, -2 lines):
M test/mjsunit/mjsunit.js
Index: test/mjsunit/mjsunit.js
diff --git a/test/mjsunit/mjsunit.js b/test/mjsunit/mjsunit.js
index
043027908853d0dfd9169553dae00a19a3efde55..00fd03bea2cb5d09295bdab8ae0a9035ce5695fa
100644
--- a/test/mjsunit/mjsunit.js
+++ b/test/mjsunit/mjsunit.js
@@ -239,7 +239,7 @@ var assertUnoptimized;
}
assertSame = function assertSame(expected, found, name_opt) {
- checkArity(arguments, 2, "assertSame");
+// checkArity(arguments, 2, "assertSame");
// TODO(mstarzinger): We should think about using Harmony's egal
operator
// or the function equivalent Object.is() here.
@@ -253,7 +253,7 @@ var assertUnoptimized;
assertEquals = function assertEquals(expected, found, name_opt) {
- checkArity(arguments, 2, "assertEquals");
+// checkArity(arguments, 2, "assertEquals");
if (!deepEquals(found, expected)) {
fail(PrettyPrint(expected), found, name_opt);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
