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

Unified Diff: runtime/vm/flow_graph_builder.cc

Issue 12518009: Remove virtual functions on class InliningContext. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/flow_graph_builder.h ('k') | runtime/vm/flow_graph_inliner.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/flow_graph_builder.cc
diff --git a/runtime/vm/flow_graph_builder.cc b/runtime/vm/flow_graph_builder.cc
index d1597f8f389ec5a9d4690e9aaa3ccea8474be502..73d05e167fda46bbf61a80c65ef5cea9ee84ed95 100644
--- a/runtime/vm/flow_graph_builder.cc
+++ b/runtime/vm/flow_graph_builder.cc
@@ -62,11 +62,6 @@ void FlowGraphBuilder::AddCatchEntry(TargetEntryInstr* entry) {
}
-InliningContext* InliningContext::Create(Definition* call) {
- return new ValueInliningContext();
-}
-
-
void InliningContext::PrepareGraphs(FlowGraph* caller_graph,
Definition* call,
FlowGraph* callee_graph) {
@@ -97,18 +92,18 @@ void InliningContext::PrepareGraphs(FlowGraph* caller_graph,
}
-void ValueInliningContext::AddExit(ReturnInstr* exit) {
+void InliningContext::AddExit(ReturnInstr* exit) {
Data data = { NULL, exit };
exits_.Add(data);
}
-int ValueInliningContext::LowestBlockIdFirst(const Data* a, const Data* b) {
+int InliningContext::LowestBlockIdFirst(const Data* a, const Data* b) {
return (a->exit_block->block_id() - b->exit_block->block_id());
}
-void ValueInliningContext::SortExits() {
+void InliningContext::SortExits() {
// Assign block entries here because we did not necessarily know them when
// the return exit was added to the array.
for (int i = 0; i < exits_.length(); ++i) {
@@ -118,9 +113,9 @@ void ValueInliningContext::SortExits() {
}
-void ValueInliningContext::ReplaceCall(FlowGraph* caller_graph,
- Definition* call,
- FlowGraph* callee_graph) {
+void InliningContext::ReplaceCall(FlowGraph* caller_graph,
+ Definition* call,
+ FlowGraph* callee_graph) {
ASSERT(call->previous() != NULL);
ASSERT(call->next() != NULL);
PrepareGraphs(caller_graph, call, callee_graph);
« no previous file with comments | « runtime/vm/flow_graph_builder.h ('k') | runtime/vm/flow_graph_inliner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698