Remove support for "loads and stores to global vars through property cell shortcuts inst…

src/compiler/js-operator.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/compiler/js-operator.h"
 
 #include <limits>
 
 #include "src/base/lazy-instance.h"
 #include "src/compiler/opcodes.h"
@@ skipping 251 matching lines @@
 const LoadNamedParameters& LoadNamedParametersOf(const Operator* op) {
   DCHECK_EQ(IrOpcode::kJSLoadNamed, op->opcode());
   return OpParameter<LoadNamedParameters>(op);
 }
 
 
 bool operator==(LoadGlobalParameters const& lhs,
                 LoadGlobalParameters const& rhs) {
   return lhs.name().location() == rhs.name().location() &&
          lhs.feedback() == rhs.feedback() &&
-         lhs.typeof_mode() == rhs.typeof_mode() &&
-         lhs.slot_index() == rhs.slot_index();
+         lhs.typeof_mode() == rhs.typeof_mode();
 }
 
 
 bool operator!=(LoadGlobalParameters const& lhs,
                 LoadGlobalParameters const& rhs) {
   return !(lhs == rhs);
 }
 
 
 size_t hash_value(LoadGlobalParameters const& p) {
-  return base::hash_combine(p.name().location(), p.typeof_mode(),
-                            p.slot_index());
+  return base::hash_combine(p.name().location(), p.typeof_mode());
 }
 
 
 std::ostream& operator<<(std::ostream& os, LoadGlobalParameters const& p) {
-  return os << Brief(*p.name()) << ", " << p.typeof_mode()
-            << ", slot: " << p.slot_index();
+  return os << Brief(*p.name()) << ", " << p.typeof_mode();
 }
 
 
 const LoadGlobalParameters& LoadGlobalParametersOf(const Operator* op) {
   DCHECK_EQ(IrOpcode::kJSLoadGlobal, op->opcode());
   return OpParameter<LoadGlobalParameters>(op);
 }
 
 
 bool operator==(StoreGlobalParameters const& lhs,
                 StoreGlobalParameters const& rhs) {
   return lhs.language_mode() == rhs.language_mode() &&
          lhs.name().location() == rhs.name().location() &&
-         lhs.feedback() == rhs.feedback() &&
-         lhs.slot_index() == rhs.slot_index();
+         lhs.feedback() == rhs.feedback();
 }
 
 
 bool operator!=(StoreGlobalParameters const& lhs,
                 StoreGlobalParameters const& rhs) {
   return !(lhs == rhs);
 }
 
 
 size_t hash_value(StoreGlobalParameters const& p) {
   return base::hash_combine(p.language_mode(), p.name().location(),
-                            p.feedback(), p.slot_index());
+                            p.feedback());
 }
 
 
 std::ostream& operator<<(std::ostream& os, StoreGlobalParameters const& p) {
-  return os << p.language_mode() << ", " << Brief(*p.name())
-            << ", slot: " << p.slot_index();
+  return os << p.language_mode() << ", " << Brief(*p.name());
 }
 
 
 const StoreGlobalParameters& StoreGlobalParametersOf(const Operator* op) {
   DCHECK_EQ(IrOpcode::kJSStoreGlobal, op->opcode());
   return OpParameter<StoreGlobalParameters>(op);
 }
 
 
 bool operator==(StoreNamedParameters const& lhs,
@@ skipping 315 matching lines @@
   return new (zone()) Operator1<LanguageMode>(               // --
       IrOpcode::kJSDeleteProperty, Operator::kNoProperties,  // opcode
       "JSDeleteProperty",                                    // name
       2, 1, 1, 1, 1, 2,                                      // counts
       language_mode);                                        // parameter
 }
 
 
 const Operator* JSOperatorBuilder::LoadGlobal(const Handle<Name>& name,
                                               const VectorSlotPair& feedback,
-                                              TypeofMode typeof_mode,
-                                              int slot_index) {
-  LoadGlobalParameters parameters(name, feedback, typeof_mode, slot_index);
+                                              TypeofMode typeof_mode) {
+  LoadGlobalParameters parameters(name, feedback, typeof_mode);
   return new (zone()) Operator1<LoadGlobalParameters>(   // --
       IrOpcode::kJSLoadGlobal, Operator::kNoProperties,  // opcode
       "JSLoadGlobal",                                    // name
-      3, 1, 1, 1, 1, 2,                                  // counts
+      1, 1, 1, 1, 1, 2,                                  // counts
       parameters);                                       // parameter
 }
 
 
 const Operator* JSOperatorBuilder::StoreGlobal(LanguageMode language_mode,
                                                const Handle<Name>& name,
-                                               const VectorSlotPair& feedback,
-                                               int slot_index) {
-  StoreGlobalParameters parameters(language_mode, feedback, name, slot_index);
+                                               const VectorSlotPair& feedback) {
+  StoreGlobalParameters parameters(language_mode, feedback, name);
   return new (zone()) Operator1<StoreGlobalParameters>(   // --
       IrOpcode::kJSStoreGlobal, Operator::kNoProperties,  // opcode
       "JSStoreGlobal",                                    // name
-      4, 1, 1, 0, 1, 2,                                   // counts
+      2, 1, 1, 0, 1, 2,                                   // counts
       parameters);                                        // parameter
 }
 
 
 const Operator* JSOperatorBuilder::LoadContext(size_t depth, size_t index,
                                                bool immutable) {
   ContextAccess access(depth, index, immutable);
   return new (zone()) Operator1<ContextAccess>(  // --
       IrOpcode::kJSLoadContext,                  // opcode
       Operator::kNoWrite | Operator::kNoThrow,   // flags
@@ skipping 117 matching lines @@
                                 Handle<ScopeInfo>::hash>(         // --
       IrOpcode::kJSCreateScriptContext, Operator::kNoProperties,  // opcode
       "JSCreateScriptContext",                                    // name
       1, 1, 1, 1, 1, 2,                                           // counts
       scpope_info);                                               // parameter
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8