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

Side by Side Diff: src/compiler/simplified-lowering.cc

Issue 2073683002: [turbofan] The Check and Checked operators don't produce control. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 6 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
« no previous file with comments | « src/compiler/representation-change.cc ('k') | src/compiler/simplified-operator.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/simplified-lowering.h" 5 #include "src/compiler/simplified-lowering.h"
6 6
7 #include <limits> 7 #include <limits>
8 8
9 #include "src/address-map.h" 9 #include "src/address-map.h"
10 #include "src/base/bits.h" 10 #include "src/base/bits.h"
(...skipping 1020 matching lines...) Expand 10 before | Expand all | Expand 10 after
1031 DCHECK(NodeProperties::IsValueEdge(edge)); 1031 DCHECK(NodeProperties::IsValueEdge(edge));
1032 } 1032 }
1033 } 1033 }
1034 } 1034 }
1035 1035
1036 void ChangeToInt32OverflowOp(Node* node, const Operator* op) { 1036 void ChangeToInt32OverflowOp(Node* node, const Operator* op) {
1037 Node* effect = NodeProperties::GetEffectInput(node); 1037 Node* effect = NodeProperties::GetEffectInput(node);
1038 Node* control = NodeProperties::GetControlInput(node); 1038 Node* control = NodeProperties::GetControlInput(node);
1039 Node* arith = graph()->NewNode(op, node->InputAt(0), node->InputAt(1)); 1039 Node* arith = graph()->NewNode(op, node->InputAt(0), node->InputAt(1));
1040 Node* overflow = graph()->NewNode(common()->Projection(1), arith); 1040 Node* overflow = graph()->NewNode(common()->Projection(1), arith);
1041 control = effect = 1041 effect =
1042 graph()->NewNode(simplified()->CheckIf(), overflow, effect, control); 1042 graph()->NewNode(simplified()->CheckIf(), overflow, effect, control);
1043 1043
1044 Node* value = graph()->NewNode(common()->Projection(0), arith); 1044 Node* value = graph()->NewNode(common()->Projection(0), arith);
1045 ReplaceEffectControlUses(node, effect, control); 1045 ReplaceEffectControlUses(node, effect, control);
1046 DeferReplacement(node, value); 1046 DeferReplacement(node, value);
1047 } 1047 }
1048 1048
1049 void VisitSpeculativeAdditiveOp(Node* node, Truncation truncation, 1049 void VisitSpeculativeAdditiveOp(Node* node, Truncation truncation,
1050 SimplifiedLowering* lowering) { 1050 SimplifiedLowering* lowering) {
1051 if (BothInputsAre(node, Type::Signed32()) && 1051 if (BothInputsAre(node, Type::Signed32()) &&
(...skipping 1835 matching lines...) Expand 10 before | Expand all | Expand 10 after
2887 isolate(), graph()->zone(), callable.descriptor(), 0, flags, 2887 isolate(), graph()->zone(), callable.descriptor(), 0, flags,
2888 Operator::kNoProperties); 2888 Operator::kNoProperties);
2889 to_number_operator_.set(common()->Call(desc)); 2889 to_number_operator_.set(common()->Call(desc));
2890 } 2890 }
2891 return to_number_operator_.get(); 2891 return to_number_operator_.get();
2892 } 2892 }
2893 2893
2894 } // namespace compiler 2894 } // namespace compiler
2895 } // namespace internal 2895 } // namespace internal
2896 } // namespace v8 2896 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/representation-change.cc ('k') | src/compiler/simplified-operator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698