Always wrap AllocationSiteContext::current() in a new handle in Crankshaft.

src/allocation-site-scopes.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_ALLOCATION_SITE_SCOPES_H_
 #define V8_ALLOCATION_SITE_SCOPES_H_
 
 #include "src/ast.h"
 #include "src/handles.h"
 #include "src/objects.h"
@@ skipping 57 matching lines @@
         top_site_(site),
         activated_(activated) { }
 
   inline Handle<AllocationSite> EnterNewScope() {
     if (top().is_null()) {
       InitializeTraversal(top_site_);
     } else {
       // Advance current site
       Object* nested_site = current()->nested_site();
       // Something is wrong if we advance to the end of the list here.
-      DCHECK(nested_site->IsAllocationSite());

[titzer, 2015/04/20 09:43:15]

This check was redundant since it's implied in the AllocationSite::cast() below.

       update_current_site(AllocationSite::cast(nested_site));
     }
     return Handle<AllocationSite>(*current(), isolate());
   }
 
   inline void ExitScope(Handle<AllocationSite> scope_site,
                         Handle<JSObject> object) {
     // This assert ensures that we are pointing at the right sub-object in a
     // recursive walk of a nested literal.
     DCHECK(object.is_null() || *object == scope_site->transition_info());
   }
 
   bool ShouldCreateMemento(Handle<JSObject> object);
 
  private:
   Handle<AllocationSite> top_site_;
   bool activated_;
 };
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_ALLOCATION_SITE_SCOPES_H_