Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(802)

Unified Diff: src/builtins/builtins-array-gen.cc

Issue 2874423003: [csa] Add ToLength and ToString variants with inlined fast checks (Closed)
Patch Set: ToString Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/builtins/builtins-conversion-gen.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/builtins-array-gen.cc
diff --git a/src/builtins/builtins-array-gen.cc b/src/builtins/builtins-array-gen.cc
index abe31e6ce8f7c097f3c9919419791a288ae85912..31d92f78bb61de53d4f421dc9985320e9b4dc1b3 100644
--- a/src/builtins/builtins-array-gen.cc
+++ b/src/builtins/builtins-array-gen.cc
@@ -332,8 +332,7 @@ class ArrayBuiltinCodeStubAssembler : public CodeStubAssembler {
BIND(&not_js_array);
Node* len_property =
GetProperty(context(), o(), isolate()->factory()->length_string());
- merged_length.Bind(
- CallStub(CodeFactory::ToLength(isolate()), context(), len_property));
+ merged_length.Bind(ToLength_Inline(context(), len_property));
Goto(&has_length);
BIND(&has_length);
len_ = merged_length.value();
@@ -2181,8 +2180,7 @@ TF_BUILTIN(ArrayIteratorPrototypeNext, CodeStubAssembler) {
{
Node* length =
GetProperty(context, array, factory()->length_string());
- Callable to_length = CodeFactory::ToLength(isolate());
- var_length.Bind(CallStub(to_length, context, length));
+ var_length.Bind(ToLength_Inline(context, length));
Goto(&done);
}
« no previous file with comments | « no previous file | src/builtins/builtins-conversion-gen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698