HStoreKeyed for Smis optimized for x64 + related redundant moves of elements removed

src/elements-kind.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 149 matching lines @@
 }
 
 
 inline bool IsFastPackedElementsKind(ElementsKind kind) {
   return kind == FAST_SMI_ELEMENTS ||
       kind == FAST_DOUBLE_ELEMENTS ||
       kind == FAST_ELEMENTS;
 }
 
 
+inline bool IsFastPackedSmiElementsKind(ElementsKind kind) {

[Toon Verwaest, 2013/12/17 13:53:04]

I wouldn't add this method. Just directly compare with FAST_SMI_ELEMENTS.

[Igor Sheludko, 2013/12/18 10:01:33]

Done.

+  return kind == FAST_SMI_ELEMENTS;
+}
+
+
 inline ElementsKind GetPackedElementsKind(ElementsKind holey_kind) {
   if (holey_kind == FAST_HOLEY_SMI_ELEMENTS) {
     return FAST_SMI_ELEMENTS;
   }
   if (holey_kind == FAST_HOLEY_DOUBLE_ELEMENTS) {
     return FAST_DOUBLE_ELEMENTS;
   }
   if (holey_kind == FAST_HOLEY_ELEMENTS) {
     return FAST_ELEMENTS;
   }
@@ skipping 50 matching lines @@
     bool allow_only_packed) {
   return IsFastElementsKind(elements_kind) &&
       (elements_kind != TERMINAL_FAST_ELEMENTS_KIND &&
        (!allow_only_packed || elements_kind != FAST_ELEMENTS));
 }
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_ELEMENTS_KIND_H_