Reviewers: Mads Ager,
Description:
Remove now obsolete comment.
Now builtins only process fast case when there is no need to lift elements
from
the prototypes.
[email protected]
Please review this at http://codereview.chromium.org/890004
Affected files:
M src/builtins.cc
Index: src/builtins.cc
diff --git a/src/builtins.cc b/src/builtins.cc
index
8b089372f2df92ddfd6d9cae8c1144c86b73e753..e59dbcf29d2168080c95e4cac648ab033d3a8ee9
100644
--- a/src/builtins.cc
+++ b/src/builtins.cc
@@ -486,10 +486,6 @@ BUILTIN(ArrayUnshift) {
int len = Smi::cast(array->length())->value();
int to_add = args.length() - 1;
- // Note that we cannot quit early if to_add == 0 as
- // values should be lifted from prototype into
- // the array.
-
int new_length = len + to_add;
// Currently fixed arrays cannot grow too big, so
// we should never hit this case.
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
