Vector ICs: Turbofan vector store ic support

src/compiler/js-operator.h

 // Copyright 2013 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.
 
 #ifndef V8_COMPILER_JS_OPERATOR_H_
 #define V8_COMPILER_JS_OPERATOR_H_
 
 #include "src/runtime/runtime.h"
 #include "src/unique.h"
 
@@ skipping 256 matching lines @@
 
 std::ostream& operator<<(std::ostream&, LoadPropertyParameters const&);
 
 const LoadPropertyParameters& LoadPropertyParametersOf(const Operator* op);
 
 
 // Defines the property being stored to an object by a named store. This is
 // used as a parameter by JSStoreNamed operators.
 class StoreNamedParameters final {
  public:
-  StoreNamedParameters(LanguageMode language_mode, const Unique<Name>& name)
-      : language_mode_(language_mode), name_(name) {}
+  StoreNamedParameters(LanguageMode language_mode,
+                       const ResolvedFeedbackSlot& feedback,
+                       const Unique<Name>& name)
+      : language_mode_(language_mode), name_(name), feedback_(feedback) {}
 
   LanguageMode language_mode() const { return language_mode_; }
+  const ResolvedFeedbackSlot& feedback() const { return feedback_; }
   const Unique<Name>& name() const { return name_; }
 
  private:
   const LanguageMode language_mode_;
   const Unique<Name> name_;
+  const ResolvedFeedbackSlot feedback_;
 };
 
 bool operator==(StoreNamedParameters const&, StoreNamedParameters const&);
 bool operator!=(StoreNamedParameters const&, StoreNamedParameters const&);
 
 size_t hash_value(StoreNamedParameters const&);
 
 std::ostream& operator<<(std::ostream&, StoreNamedParameters const&);
 
 const StoreNamedParameters& StoreNamedParametersOf(const Operator* op);
 
 
+// Defines the property being loaded from an object. This is

[Michael Starzinger, 2015/06/15 14:20:11]

nit: s/loaded from an object/stored to an object/

[mvstanton, 2015/06/21 13:37:36]

Done.

+// used as a parameter by JSLoadProperty operators.

[Michael Starzinger, 2015/06/15 14:20:11]

nit: s/JSLoadProperty/JSStoreProperty/

[mvstanton, 2015/06/21 13:37:36]

thx, done.

+class StorePropertyParameters final {
+ public:
+  StorePropertyParameters(LanguageMode language_mode,
+                          const ResolvedFeedbackSlot& feedback)
+      : language_mode_(language_mode), feedback_(feedback) {}
+
+  LanguageMode language_mode() const { return language_mode_; }
+  const ResolvedFeedbackSlot& feedback() const { return feedback_; }
+
+ private:
+  const LanguageMode language_mode_;
+  const ResolvedFeedbackSlot feedback_;
+};
+
+bool operator==(StorePropertyParameters const&, StorePropertyParameters const&);
+bool operator!=(StorePropertyParameters const&, StorePropertyParameters const&);
+
+size_t hash_value(StorePropertyParameters const&);
+
+std::ostream& operator<<(std::ostream&, StorePropertyParameters const&);
+
+const StorePropertyParameters& StorePropertyParametersOf(const Operator* op);
+
+
 // Defines shared information for the closure that should be created. This is
 // used as a parameter by JSCreateClosure operators.
 class CreateClosureParameters final {
  public:
   CreateClosureParameters(Handle<SharedFunctionInfo> shared_info,
                           PretenureFlag pretenure)
       : shared_info_(shared_info), pretenure_(pretenure) {}
 
   Handle<SharedFunctionInfo> shared_info() const { return shared_info_; }
   PretenureFlag pretenure() const { return pretenure_; }
@@ skipping 58 matching lines @@
                                LanguageMode language_mode);
   const Operator* CallRuntime(Runtime::FunctionId id, size_t arity);
 
   const Operator* CallConstruct(int arguments);
 
   const Operator* LoadProperty(const ResolvedFeedbackSlot& feedback);
   const Operator* LoadNamed(const Unique<Name>& name,
                             const ResolvedFeedbackSlot& feedback,
                             ContextualMode contextual_mode = NOT_CONTEXTUAL);
 
-  const Operator* StoreProperty(LanguageMode language_mode);
+  const Operator* StoreProperty(LanguageMode language_mode,
+                                const ResolvedFeedbackSlot& feedback);
   const Operator* StoreNamed(LanguageMode language_mode,
+                             const ResolvedFeedbackSlot& feedback,
                              const Unique<Name>& name);

[Michael Starzinger, 2015/06/15 14:20:11]

nit: IMHO the name should go first or at least before the feedback.

[mvstanton, 2015/06/21 13:37:36]

Done.

 
   const Operator* DeleteProperty(LanguageMode language_mode);
 
   const Operator* HasProperty();
 
   const Operator* LoadContext(size_t depth, size_t index, bool immutable);
   const Operator* StoreContext(size_t depth, size_t index);
 
   const Operator* LoadDynamicGlobal(const Handle<String>& name,
                                     uint32_t check_bitset,
@@ skipping 28 matching lines @@
   Zone* const zone_;
 
   DISALLOW_COPY_AND_ASSIGN(JSOperatorBuilder);
 };
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_COMPILER_JS_OPERATOR_H_