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

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

Issue 2766383003: [turbofan] escape analysis: revisit of uses afer FinishRegion marked as escaped (Closed)
Patch Set: repro 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
« no previous file with comments | « no previous file | test/mjsunit/compiler/escape-analysis-15.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 863 matching lines...) Expand 10 before | Expand all | Expand 10 after
874 } 874 }
875 875
876 void EscapeStatusAnalysis::ProcessFinishRegion(Node* node) { 876 void EscapeStatusAnalysis::ProcessFinishRegion(Node* node) {
877 DCHECK_EQ(node->opcode(), IrOpcode::kFinishRegion); 877 DCHECK_EQ(node->opcode(), IrOpcode::kFinishRegion);
878 if (!HasEntry(node)) { 878 if (!HasEntry(node)) {
879 status_[node->id()] |= kTracked; 879 status_[node->id()] |= kTracked;
880 RevisitUses(node); 880 RevisitUses(node);
881 } 881 }
882 if (CheckUsesForEscape(node, true)) { 882 if (CheckUsesForEscape(node, true)) {
883 RevisitInputs(node); 883 RevisitInputs(node);
884 RevisitUses(node);
884 } 885 }
885 } 886 }
886 887
887 void EscapeStatusAnalysis::DebugPrint() { 888 void EscapeStatusAnalysis::DebugPrint() {
888 for (NodeId id = 0; id < status_.size(); id++) { 889 for (NodeId id = 0; id < status_.size(); id++) {
889 if (status_[id] & kTracked) { 890 if (status_[id] & kTracked) {
890 PrintF("Node #%d is %s\n", id, 891 PrintF("Node #%d is %s\n", id,
891 (status_[id] & kEscaped) ? "escaping" : "virtual"); 892 (status_[id] & kEscaped) ? "escaping" : "virtual");
892 } 893 }
893 } 894 }
(...skipping 904 matching lines...) Expand 10 before | Expand all | Expand 10 after
1798 } 1799 }
1799 } 1800 }
1800 return false; 1801 return false;
1801 } 1802 }
1802 1803
1803 Graph* EscapeAnalysis::graph() const { return status_analysis_->graph(); } 1804 Graph* EscapeAnalysis::graph() const { return status_analysis_->graph(); }
1804 1805
1805 } // namespace compiler 1806 } // namespace compiler
1806 } // namespace internal 1807 } // namespace internal
1807 } // namespace v8 1808 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | test/mjsunit/compiler/escape-analysis-15.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698