Additional work to get array literal allocation tracking working, even with --always-opt

src/ic.cc

Index: src/ic.cc
diff --git a/src/ic.cc b/src/ic.cc
index 5c4150bf40208868635dedac205e17743b186ef6..76e02bdcc9eaaebca47b11cb2ff56c86f5fc4fbe 100644
--- a/src/ic.cc
+++ b/src/ic.cc
@@ -1852,6 +1852,11 @@ Handle<Code> KeyedStoreIC::ComputePolymorphicStub(
     Handle<Code> cached_stub;
     Handle<Map> transitioned_map =
         receiver_map->FindTransitionedMap(receiver_maps);
+
+    // TODO(mvstanton): we should set the transitioned map to always be

[Toon Verwaest, 2013/01/16 10:53:42]

Should probably clarify that we are talking about pessimistic elements
transitions.

[mvstanton, 2013/01/16 13:01:56]

Done.

+    // null, and quit doing transitions on key stores. Right now though
+    // if I do this there are some serious performance issues.
+
     if (!transitioned_map.is_null()) {
       cached_stub = ElementsTransitionAndStoreStub(
           receiver_map->elements_kind(),  // original elements_kind