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

Unified Diff: test/cctest/compiler/structured-machine-assembler.cc

Issue 543763002: Remove overly complex MachineNodeFactory. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/cctest/compiler/structured-machine-assembler.h ('k') | tools/gyp/v8.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/compiler/structured-machine-assembler.cc
diff --git a/test/cctest/compiler/structured-machine-assembler.cc b/test/cctest/compiler/structured-machine-assembler.cc
index 2bb1e8b5a6f63500c13b1c6fed557e65f0d197fd..66c408a815af5fca1ee14c0b1141d89a014c3880 100644
--- a/test/cctest/compiler/structured-machine-assembler.cc
+++ b/test/cctest/compiler/structured-machine-assembler.cc
@@ -18,44 +18,10 @@ void Variable::Set(Node* value) const { smasm_->SetVariable(offset_, value); }
StructuredMachineAssembler::StructuredMachineAssembler(
Graph* graph, MachineSignature* machine_sig, MachineType word)
- : GraphBuilder(graph),
- schedule_(new (zone()) Schedule(zone())),
- machine_(zone(), word),
- common_(zone()),
- machine_sig_(machine_sig),
- call_descriptor_(
- Linkage::GetSimplifiedCDescriptor(graph->zone(), machine_sig)),
- parameters_(NULL),
+ : RawMachineAssembler(graph, machine_sig, word),
current_environment_(new (zone())
Environment(zone(), schedule()->start(), false)),
- number_of_variables_(0) {
- int param_count = static_cast<int>(parameter_count());
- Node* s = graph->NewNode(common_.Start(param_count));
- graph->SetStart(s);
- if (parameter_count() == 0) return;
- parameters_ = zone()->NewArray<Node*>(param_count);
- for (size_t i = 0; i < parameter_count(); ++i) {
- parameters_[i] =
- NewNode(common()->Parameter(static_cast<int>(i)), graph->start());
- }
-}
-
-
-Schedule* StructuredMachineAssembler::Export() {
- // Compute the correct codegen order.
- DCHECK(schedule_->rpo_order()->empty());
- Scheduler::ComputeSpecialRPO(schedule_);
- // Invalidate MachineAssembler.
- Schedule* schedule = schedule_;
- schedule_ = NULL;
- return schedule;
-}
-
-
-Node* StructuredMachineAssembler::Parameter(size_t index) {
- DCHECK(index < parameter_count());
- return parameters_[index];
-}
+ number_of_variables_(0) {}
Node* StructuredMachineAssembler::MakeNode(Operator* op, int input_count,
« no previous file with comments | « test/cctest/compiler/structured-machine-assembler.h ('k') | tools/gyp/v8.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698