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

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

Issue 23788012: Fix an off-by-one error in deoptimization compression. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added changes to all platform files. Created 7 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 | Annotate | Revision Log
« no previous file with comments | « runtime/vm/deopt_instructions.cc ('k') | runtime/vm/flow_graph_compiler.cc » ('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 #ifndef VM_FLOW_GRAPH_COMPILER_H_ 5 #ifndef VM_FLOW_GRAPH_COMPILER_H_
6 #define VM_FLOW_GRAPH_COMPILER_H_ 6 #define VM_FLOW_GRAPH_COMPILER_H_
7 7
8 #include "vm/allocation.h" 8 #include "vm/allocation.h"
9 #include "vm/assembler.h" 9 #include "vm/assembler.h"
10 #include "vm/code_descriptors.h" 10 #include "vm/code_descriptors.h"
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 DeoptReasonId reason, 113 DeoptReasonId reason,
114 Environment* deopt_env) 114 Environment* deopt_env)
115 : pc_offset_(-1), 115 : pc_offset_(-1),
116 deopt_id_(deopt_id), 116 deopt_id_(deopt_id),
117 reason_(reason), 117 reason_(reason),
118 deopt_env_(deopt_env) { 118 deopt_env_(deopt_env) {
119 ASSERT(deopt_env != NULL); 119 ASSERT(deopt_env != NULL);
120 } 120 }
121 121
122 RawDeoptInfo* CreateDeoptInfo(FlowGraphCompiler* compiler, 122 RawDeoptInfo* CreateDeoptInfo(FlowGraphCompiler* compiler,
123 DeoptInfoBuilder* builder); 123 DeoptInfoBuilder* builder,
124 const Array& deopt_table);
124 125
125 126
126 // No code needs to be generated. 127 // No code needs to be generated.
127 virtual void GenerateCode(FlowGraphCompiler* compiler, intptr_t stub_ix) {} 128 virtual void GenerateCode(FlowGraphCompiler* compiler, intptr_t stub_ix) {}
128 129
129 intptr_t pc_offset() const { return pc_offset_; } 130 intptr_t pc_offset() const { return pc_offset_; }
130 void set_pc_offset(intptr_t offset) { pc_offset_ = offset; } 131 void set_pc_offset(intptr_t offset) { pc_offset_ = offset; }
131 132
132 intptr_t deopt_id() const { return deopt_id_; } 133 intptr_t deopt_id() const { return deopt_id_; }
133 DeoptReasonId reason() const { return reason_; } 134 DeoptReasonId reason() const { return reason_; }
(...skipping 469 matching lines...) Expand 10 before | Expand all | Expand 10 after
603 // that should be used when deoptimizing we store it in this variable. 604 // that should be used when deoptimizing we store it in this variable.
604 // In future AddDeoptStub should be moved out of the instruction template. 605 // In future AddDeoptStub should be moved out of the instruction template.
605 Environment* pending_deoptimization_env_; 606 Environment* pending_deoptimization_env_;
606 607
607 DISALLOW_COPY_AND_ASSIGN(FlowGraphCompiler); 608 DISALLOW_COPY_AND_ASSIGN(FlowGraphCompiler);
608 }; 609 };
609 610
610 } // namespace dart 611 } // namespace dart
611 612
612 #endif // VM_FLOW_GRAPH_COMPILER_H_ 613 #endif // VM_FLOW_GRAPH_COMPILER_H_
OLDNEW
« no previous file with comments | « runtime/vm/deopt_instructions.cc ('k') | runtime/vm/flow_graph_compiler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698