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

Side by Side Diff: src/compiler/js-builtin-reducer.cc

Issue 2722483003: [turbofan] Introduce dedicated ObjectIsNaN operator. (Closed)
Patch Set: Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/compiler/js-builtin-reducer.h" 5 #include "src/compiler/js-builtin-reducer.h"
6 6
7 #include "src/base/bits.h" 7 #include "src/base/bits.h"
8 #include "src/code-factory.h" 8 #include "src/code-factory.h"
9 #include "src/compilation-dependencies.h" 9 #include "src/compilation-dependencies.h"
10 #include "src/compiler/access-builder.h" 10 #include "src/compiler/access-builder.h"
(...skipping 1397 matching lines...) Expand 10 before | Expand all | Expand 10 after
1408 Node* value = graph()->NewNode(simplified()->NumberEqual(), diff, 1408 Node* value = graph()->NewNode(simplified()->NumberEqual(), diff,
1409 jsgraph()->ZeroConstant()); 1409 jsgraph()->ZeroConstant());
1410 return Replace(value); 1410 return Replace(value);
1411 } 1411 }
1412 return NoChange(); 1412 return NoChange();
1413 } 1413 }
1414 1414
1415 // ES6 section 20.1.2.4 Number.isNaN ( number ) 1415 // ES6 section 20.1.2.4 Number.isNaN ( number )
1416 Reduction JSBuiltinReducer::ReduceNumberIsNaN(Node* node) { 1416 Reduction JSBuiltinReducer::ReduceNumberIsNaN(Node* node) {
1417 JSCallReduction r(node); 1417 JSCallReduction r(node);
1418 if (r.InputsMatchOne(Type::Number())) { 1418 // Number.isNaN(a:number) -> ObjectIsNaN(a)
1419 // Number.isNaN(a:number) -> BooleanNot(NumberEqual(a, a)) 1419 Node* input = r.GetJSCallInput(0);
1420 Node* input = r.GetJSCallInput(0); 1420 Node* value = graph()->NewNode(simplified()->ObjectIsNaN(), input);
1421 Node* check = graph()->NewNode(simplified()->NumberEqual(), input, input); 1421 return Replace(value);
1422 Node* value = graph()->NewNode(simplified()->BooleanNot(), check);
1423 return Replace(value);
1424 }
1425 return NoChange();
1426 } 1422 }
1427 1423
1428 // ES6 section 20.1.2.5 Number.isSafeInteger ( number ) 1424 // ES6 section 20.1.2.5 Number.isSafeInteger ( number )
1429 Reduction JSBuiltinReducer::ReduceNumberIsSafeInteger(Node* node) { 1425 Reduction JSBuiltinReducer::ReduceNumberIsSafeInteger(Node* node) {
1430 JSCallReduction r(node); 1426 JSCallReduction r(node);
1431 if (r.InputsMatchOne(type_cache_.kSafeInteger)) { 1427 if (r.InputsMatchOne(type_cache_.kSafeInteger)) {
1432 // Number.isInteger(x:safe-integer) -> #true 1428 // Number.isInteger(x:safe-integer) -> #true
1433 Node* value = jsgraph()->TrueConstant(); 1429 Node* value = jsgraph()->TrueConstant();
1434 return Replace(value); 1430 return Replace(value);
1435 } 1431 }
(...skipping 742 matching lines...) Expand 10 before | Expand all | Expand 10 after
2178 return jsgraph()->simplified(); 2174 return jsgraph()->simplified();
2179 } 2175 }
2180 2176
2181 JSOperatorBuilder* JSBuiltinReducer::javascript() const { 2177 JSOperatorBuilder* JSBuiltinReducer::javascript() const {
2182 return jsgraph()->javascript(); 2178 return jsgraph()->javascript();
2183 } 2179 }
2184 2180
2185 } // namespace compiler 2181 } // namespace compiler
2186 } // namespace internal 2182 } // namespace internal
2187 } // namespace v8 2183 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698