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

Unified Diff: src/compiler/js-native-context-specialization.h

Issue 1417043006: [turbofan] Split JSGlobalObjectSpecialization into separate class. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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 | « src/compiler/js-inlining.cc ('k') | src/compiler/js-native-context-specialization.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/js-native-context-specialization.h
diff --git a/src/compiler/js-native-context-specialization.h b/src/compiler/js-native-context-specialization.h
index f87edca69dbb2e0fc5a26b689d347538d70fddea..89adcce601ca0482174a93c913b1581bc96ca7f9 100644
--- a/src/compiler/js-native-context-specialization.h
+++ b/src/compiler/js-native-context-specialization.h
@@ -8,7 +8,6 @@
#include "src/base/flags.h"
#include "src/compiler/access-info.h"
#include "src/compiler/graph-reducer.h"
-#include "src/compiler/simplified-operator.h"
namespace v8 {
namespace internal {
@@ -27,6 +26,7 @@ class CommonOperatorBuilder;
class JSGraph;
class JSOperatorBuilder;
class MachineOperatorBuilder;
+class SimplifiedOperatorBuilder;
// Specializes a given JSGraph to a given native context, potentially constant
@@ -43,7 +43,7 @@ class JSNativeContextSpecialization final : public AdvancedReducer {
typedef base::Flags<Flag> Flags;
JSNativeContextSpecialization(Editor* editor, JSGraph* jsgraph, Flags flags,
- Handle<JSGlobalObject> global_object,
+ Handle<Context> native_context,
CompilationDependencies* dependencies,
Zone* zone);
@@ -51,20 +51,11 @@ class JSNativeContextSpecialization final : public AdvancedReducer {
private:
Reduction ReduceJSCallFunction(Node* node);
- Reduction ReduceJSLoadGlobal(Node* node);
- Reduction ReduceJSStoreGlobal(Node* node);
Reduction ReduceJSLoadNamed(Node* node);
Reduction ReduceJSStoreNamed(Node* node);
Reduction ReduceJSLoadProperty(Node* node);
Reduction ReduceJSStoreProperty(Node* node);
- Reduction Replace(Node* node, Node* value, Node* effect = nullptr,
- Node* control = nullptr) {
- ReplaceWithValue(node, value, effect, control);
- return Changed(value);
- }
- Reduction Replace(Node* node, Handle<Object> value);
-
Reduction ReduceElementAccess(Node* node, Node* index, Node* value,
MapHandleList const& receiver_maps,
AccessMode access_mode,
@@ -79,10 +70,6 @@ class JSNativeContextSpecialization final : public AdvancedReducer {
LanguageMode language_mode,
Node* index = nullptr);
- struct ScriptContextTableLookupResult;
- bool LookupInScriptContextTable(Handle<Name> name,
- ScriptContextTableLookupResult* result);
-
// Adds stability dependencies on all prototypes of every class in
// {receiver_type} up to (and including) the {holder}.
void AssumePrototypesStable(Type* receiver_type, Handle<JSObject> holder);
@@ -100,7 +87,6 @@ class JSNativeContextSpecialization final : public AdvancedReducer {
SimplifiedOperatorBuilder* simplified() const;
MachineOperatorBuilder* machine() const;
Flags flags() const { return flags_; }
- Handle<JSGlobalObject> global_object() const { return global_object_; }
Handle<Context> native_context() const { return native_context_; }
CompilationDependencies* dependencies() const { return dependencies_; }
Zone* zone() const { return zone_; }
@@ -108,7 +94,6 @@ class JSNativeContextSpecialization final : public AdvancedReducer {
JSGraph* const jsgraph_;
Flags const flags_;
- Handle<JSGlobalObject> global_object_;
Handle<Context> native_context_;
CompilationDependencies* const dependencies_;
Zone* const zone_;
« no previous file with comments | « src/compiler/js-inlining.cc ('k') | src/compiler/js-native-context-specialization.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698