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

Side by Side Diff: runtime/vm/flow_graph_compiler.cc

Issue 10857016: Refactored FlowGraphBuilder into a separate FlowGraph representation. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Revision based on Kevin's review. Created 8 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « runtime/vm/flow_graph_builder.cc ('k') | runtime/vm/flow_graph_compiler_arm.h » ('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 (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_XXX. 5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_XXX.
6 6
7 #include "vm/flow_graph_compiler.h" 7 #include "vm/flow_graph_compiler.h"
8 8
9 #include "vm/dart_entry.h" 9 #include "vm/dart_entry.h"
10 #include "vm/debugger.h" 10 #include "vm/debugger.h"
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 builder.AddCopy(deoptimization_env_->LocationAt(i), *values[i], slot_ix++); 69 builder.AddCopy(deoptimization_env_->LocationAt(i), *values[i], slot_ix++);
70 } 70 }
71 71
72 const DeoptInfo& deopt_info = DeoptInfo::Handle(builder.CreateDeoptInfo()); 72 const DeoptInfo& deopt_info = DeoptInfo::Handle(builder.CreateDeoptInfo());
73 return deopt_info.raw(); 73 return deopt_info.raw();
74 } 74 }
75 75
76 76
77 FlowGraphCompiler::FlowGraphCompiler( 77 FlowGraphCompiler::FlowGraphCompiler(
78 Assembler* assembler, 78 Assembler* assembler,
79 const ParsedFunction& parsed_function, 79 const FlowGraph& flow_graph,
80 const GrowableArray<BlockEntryInstr*>& block_order,
81 bool is_optimizing, 80 bool is_optimizing,
82 bool is_ssa, 81 bool is_ssa,
83 bool is_leaf) 82 bool is_leaf)
84 : assembler_(assembler), 83 : assembler_(assembler),
85 parsed_function_(parsed_function), 84 parsed_function_(flow_graph.parsed_function()),
86 block_order_(block_order), 85 block_order_(flow_graph.reverse_postorder()),
87 current_block_(NULL), 86 current_block_(NULL),
88 exception_handlers_list_(NULL), 87 exception_handlers_list_(NULL),
89 pc_descriptors_list_(NULL), 88 pc_descriptors_list_(NULL),
90 stackmap_table_builder_(is_ssa ? new StackmapTableBuilder() : NULL), 89 stackmap_table_builder_(is_ssa ? new StackmapTableBuilder() : NULL),
91 block_info_(block_order.length()), 90 block_info_(block_order_.length()),
92 deopt_stubs_(), 91 deopt_stubs_(),
93 object_table_(GrowableObjectArray::Handle(GrowableObjectArray::New())), 92 object_table_(GrowableObjectArray::Handle(GrowableObjectArray::New())),
94 is_optimizing_(is_optimizing), 93 is_optimizing_(is_optimizing),
95 is_ssa_(is_ssa), 94 is_ssa_(is_ssa),
96 is_dart_leaf_(is_leaf), 95 is_dart_leaf_(is_leaf),
97 bool_true_(Bool::ZoneHandle(Bool::True())), 96 bool_true_(Bool::ZoneHandle(Bool::True())),
98 bool_false_(Bool::ZoneHandle(Bool::False())), 97 bool_false_(Bool::ZoneHandle(Bool::False())),
99 double_class_(Class::ZoneHandle( 98 double_class_(Class::ZoneHandle(
100 Isolate::Current()->object_store()->double_class())), 99 Isolate::Current()->object_store()->double_class())),
101 frame_register_allocator_(this, is_optimizing, is_ssa), 100 frame_register_allocator_(this, is_optimizing, is_ssa),
(...skipping 773 matching lines...) Expand 10 before | Expand all | Expand 10 after
875 return; 874 return;
876 } 875 }
877 } 876 }
878 877
879 // This move is not blocked. 878 // This move is not blocked.
880 EmitMove(index); 879 EmitMove(index);
881 } 880 }
882 881
883 882
884 } // namespace dart 883 } // namespace dart
OLDNEW
« no previous file with comments | « runtime/vm/flow_graph_builder.cc ('k') | runtime/vm/flow_graph_compiler_arm.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698