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

Unified Diff: src/compiler/js-generic-lowering.h

Issue 565893002: Cleanup and simplify TurboFan generic lowering. (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 | « no previous file | src/compiler/js-generic-lowering.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/js-generic-lowering.h
diff --git a/src/compiler/js-generic-lowering.h b/src/compiler/js-generic-lowering.h
index da554ca864cb3277a110fbd875209e8e789e5151..400f8062a5c1466cc8ec13b50dd4820bd189bd50 100644
--- a/src/compiler/js-generic-lowering.h
+++ b/src/compiler/js-generic-lowering.h
@@ -13,7 +13,6 @@
#include "src/compiler/graph-reducer.h"
#include "src/compiler/js-graph.h"
#include "src/compiler/opcodes.h"
-#include "src/unique.h"
namespace v8 {
namespace internal {
@@ -27,15 +26,14 @@ class Linkage;
// Lowers JS-level operators to runtime and IC calls in the "generic" case.
class JSGenericLowering : public Reducer {
public:
- JSGenericLowering(CompilationInfo* info, JSGraph* graph,
- MachineOperatorBuilder* machine);
+ JSGenericLowering(CompilationInfo* info, JSGraph* graph);
virtual ~JSGenericLowering() {}
virtual Reduction Reduce(Node* node);
protected:
-// Dispatched depending on opcode.
-#define DECLARE_LOWER(x) Node* Lower##x(Node* node);
+#define DECLARE_LOWER(x) void Lower##x(Node* node);
+ // Dispatched depending on opcode.
ALL_OP_LIST(DECLARE_LOWER)
#undef DECLARE_LOWER
@@ -52,8 +50,7 @@ class JSGenericLowering : public Reducer {
// Helpers to replace existing nodes with a generic call.
void ReplaceWithCompareIC(Node* node, Token::Value token, bool pure);
- void ReplaceWithStubCall(Node* node, Callable callable,
- CallDescriptor::Flags flags);
+ void ReplaceWithStubCall(Node* node, Callable c, CallDescriptor::Flags flags);
mvstanton 2014/09/12 14:05:00 terseness is better than redundancy :).
void ReplaceWithBuiltinCall(Node* node, Builtins::JavaScript id, int args);
void ReplaceWithRuntimeCall(Node* node, Runtime::FunctionId f, int args = -1);
@@ -64,13 +61,12 @@ class JSGenericLowering : public Reducer {
Linkage* linkage() const { return linkage_; }
CompilationInfo* info() const { return info_; }
CommonOperatorBuilder* common() const { return jsgraph()->common(); }
- MachineOperatorBuilder* machine() const { return machine_; }
+ MachineOperatorBuilder* machine() const { return jsgraph()->machine(); }
private:
CompilationInfo* info_;
JSGraph* jsgraph_;
Linkage* linkage_;
- MachineOperatorBuilder* machine_;
SetOncePointer<Node> centrystub_constant_;
};
« no previous file with comments | « no previous file | src/compiler/js-generic-lowering.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698