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

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

Issue 1343383003: VM: Store edge counters in one per-function array. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: clean up comments, save space in IL Instruction class. Created 5 years, 3 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
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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_ARM64. 5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_ARM64.
6 #if defined(TARGET_ARCH_ARM64) 6 #if defined(TARGET_ARCH_ARM64)
7 7
8 #include "vm/intermediate_language.h" 8 #include "vm/intermediate_language.h"
9 9
10 #include "vm/dart_entry.h" 10 #include "vm/dart_entry.h"
(...skipping 5397 matching lines...) Expand 10 before | Expand all | Expand 10 after
5408 5408
5409 LocationSummary* GotoInstr::MakeLocationSummary(Zone* zone, 5409 LocationSummary* GotoInstr::MakeLocationSummary(Zone* zone,
5410 bool opt) const { 5410 bool opt) const {
5411 return new(zone) LocationSummary(zone, 0, 0, LocationSummary::kNoCall); 5411 return new(zone) LocationSummary(zone, 0, 0, LocationSummary::kNoCall);
5412 } 5412 }
5413 5413
5414 5414
5415 void GotoInstr::EmitNativeCode(FlowGraphCompiler* compiler) { 5415 void GotoInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
5416 if (!compiler->is_optimizing()) { 5416 if (!compiler->is_optimizing()) {
5417 if (FLAG_emit_edge_counters) { 5417 if (FLAG_emit_edge_counters) {
5418 compiler->EmitEdgeCounter(); 5418 compiler->EmitEdgeCounter(block()->preorder_number());
5419 } 5419 }
5420 // Add a deoptimization descriptor for deoptimizing instructions that 5420 // Add a deoptimization descriptor for deoptimizing instructions that
5421 // may be inserted before this instruction. On ARM64 this descriptor 5421 // may be inserted before this instruction.
5422 // points after the edge counter code so that we can reuse the same
5423 // pattern matching code as at call sites, which matches backwards from
5424 // the end of the pattern.
5425 compiler->AddCurrentDescriptor(RawPcDescriptors::kDeopt, 5422 compiler->AddCurrentDescriptor(RawPcDescriptors::kDeopt,
5426 GetDeoptId(), 5423 GetDeoptId(),
5427 Scanner::kNoSourcePos); 5424 Scanner::kNoSourcePos);
5428 } 5425 }
5429 if (HasParallelMove()) { 5426 if (HasParallelMove()) {
5430 compiler->parallel_move_resolver()->EmitNativeCode(parallel_move()); 5427 compiler->parallel_move_resolver()->EmitNativeCode(parallel_move());
5431 } 5428 }
5432 5429
5433 // We can fall through if the successor is the next block in the list. 5430 // We can fall through if the successor is the next block in the list.
5434 // Otherwise, we need a jump. 5431 // Otherwise, we need a jump.
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
5626 1, 5623 1,
5627 locs()); 5624 locs());
5628 __ Drop(1); 5625 __ Drop(1);
5629 __ Pop(result); 5626 __ Pop(result);
5630 } 5627 }
5631 5628
5632 5629
5633 } // namespace dart 5630 } // namespace dart
5634 5631
5635 #endif // defined TARGET_ARCH_ARM64 5632 #endif // defined TARGET_ARCH_ARM64
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698