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

Side by Side Diff: src/compiler/escape-analysis.cc

Issue 2533263002: [turbofan] Teach escape analysis about ConvertTaggedHoleToUndefined. (Closed)
Patch Set: Created 4 years 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 | « no previous file | test/mjsunit/regress/regress-crbug-669451.js » ('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 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 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/escape-analysis.h" 5 #include "src/compiler/escape-analysis.h"
6 6
7 #include <limits> 7 #include <limits>
8 8
9 #include "src/base/flags.h" 9 #include "src/base/flags.h"
10 #include "src/bootstrapper.h" 10 #include "src/bootstrapper.h"
(...skipping 778 matching lines...) Expand 10 before | Expand all | Expand 10 after
789 if (!IsAllocation(rep) && SetEscaped(rep)) { 789 if (!IsAllocation(rep) && SetEscaped(rep)) {
790 TRACE("Setting #%d (%s) to escaped because of use by #%d (%s)\n", 790 TRACE("Setting #%d (%s) to escaped because of use by #%d (%s)\n",
791 rep->id(), rep->op()->mnemonic(), use->id(), 791 rep->id(), rep->op()->mnemonic(), use->id(),
792 use->op()->mnemonic()); 792 use->op()->mnemonic());
793 return true; 793 return true;
794 } 794 }
795 break; 795 break;
796 case IrOpcode::kSelect: 796 case IrOpcode::kSelect:
797 // TODO(mstarzinger): The following list of operators will eventually be 797 // TODO(mstarzinger): The following list of operators will eventually be
798 // handled by the EscapeAnalysisReducer (similar to ObjectIsSmi). 798 // handled by the EscapeAnalysisReducer (similar to ObjectIsSmi).
799 case IrOpcode::kConvertTaggedHoleToUndefined:
799 case IrOpcode::kStringEqual: 800 case IrOpcode::kStringEqual:
800 case IrOpcode::kStringLessThan: 801 case IrOpcode::kStringLessThan:
801 case IrOpcode::kStringLessThanOrEqual: 802 case IrOpcode::kStringLessThanOrEqual:
802 case IrOpcode::kTypeGuard: 803 case IrOpcode::kTypeGuard:
803 case IrOpcode::kPlainPrimitiveToNumber: 804 case IrOpcode::kPlainPrimitiveToNumber:
804 case IrOpcode::kPlainPrimitiveToWord32: 805 case IrOpcode::kPlainPrimitiveToWord32:
805 case IrOpcode::kPlainPrimitiveToFloat64: 806 case IrOpcode::kPlainPrimitiveToFloat64:
806 case IrOpcode::kStringCharCodeAt: 807 case IrOpcode::kStringCharCodeAt:
807 case IrOpcode::kObjectIsCallable: 808 case IrOpcode::kObjectIsCallable:
808 case IrOpcode::kObjectIsNumber: 809 case IrOpcode::kObjectIsNumber:
(...skipping 875 matching lines...) Expand 10 before | Expand all | Expand 10 after
1684 } 1685 }
1685 } 1686 }
1686 return false; 1687 return false;
1687 } 1688 }
1688 1689
1689 Graph* EscapeAnalysis::graph() const { return status_analysis_->graph(); } 1690 Graph* EscapeAnalysis::graph() const { return status_analysis_->graph(); }
1690 1691
1691 } // namespace compiler 1692 } // namespace compiler
1692 } // namespace internal 1693 } // namespace internal
1693 } // namespace v8 1694 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | test/mjsunit/regress/regress-crbug-669451.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698