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: runtime/vm/flow_graph_compiler.cc

Issue 10536145: Fuse comparisons that are used by branches together. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 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 | Annotate | Revision Log
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/debugger.h" 9 #include "vm/debugger.h"
10 #include "vm/il_printer.h" 10 #include "vm/il_printer.h"
(...skipping 19 matching lines...) Expand all
30 bool is_optimizing) 30 bool is_optimizing)
31 : assembler_(assembler), 31 : assembler_(assembler),
32 parsed_function_(parsed_function), 32 parsed_function_(parsed_function),
33 block_order_(block_order), 33 block_order_(block_order),
34 current_block_(NULL), 34 current_block_(NULL),
35 exception_handlers_list_(NULL), 35 exception_handlers_list_(NULL),
36 pc_descriptors_list_(NULL), 36 pc_descriptors_list_(NULL),
37 stackmap_builder_(NULL), 37 stackmap_builder_(NULL),
38 block_info_(block_order.length()), 38 block_info_(block_order.length()),
39 deopt_stubs_(), 39 deopt_stubs_(),
40 is_optimizing_(is_optimizing) { 40 is_optimizing_(is_optimizing),
41 true_value_(Bool::ZoneHandle(Bool::True())),
42 false_value_(Bool::ZoneHandle(Bool::False())) {
41 ASSERT(assembler != NULL); 43 ASSERT(assembler != NULL);
42 } 44 }
43 45
44 46
45 FlowGraphCompiler::~FlowGraphCompiler() { 47 FlowGraphCompiler::~FlowGraphCompiler() {
46 // BlockInfos are zone-allocated, so their destructors are not called. 48 // BlockInfos are zone-allocated, so their destructors are not called.
47 // Verify the labels explicitly here. 49 // Verify the labels explicitly here.
48 for (int i = 0; i < block_info_.length(); ++i) { 50 for (int i = 0; i < block_info_.length(); ++i) {
49 ASSERT(!block_info_[i]->label.IsLinked()); 51 ASSERT(!block_info_[i]->label.IsLinked());
50 ASSERT(!block_info_[i]->label.HasNear()); 52 ASSERT(!block_info_[i]->label.HasNear());
(...skipping 333 matching lines...) Expand 10 before | Expand all | Expand 10 after
384 function_name, 386 function_name,
385 kNumArguments, 387 kNumArguments,
386 Array::ZoneHandle(), // No optional arguments. 388 Array::ZoneHandle(), // No optional arguments.
387 kNumArgsChecked); 389 kNumArgsChecked);
388 } 390 }
389 391
390 392
391 393
392 394
393 } // namespace dart 395 } // namespace dart
OLDNEW
« no previous file with comments | « no previous file | runtime/vm/flow_graph_compiler_ia32.h » ('j') | runtime/vm/flow_graph_compiler_ia32.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698