[TypeFeedbackVector] Root feedback vectors at function literal site.

test/cctest/test-serialize.cc

 // Copyright 2007-2010 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 244 matching lines @@
       v8::HandleScope handle_scope(v8_isolate);
       v8::Local<v8::Context>::New(v8_isolate, env)->Enter();
     }
     // Make sure all builtin scripts are cached.
     {
       HandleScope scope(isolate);
       for (int i = 0; i < Natives::GetBuiltinsCount(); i++) {
         isolate->bootstrapper()->SourceLookup<Natives>(i);
       }
     }
+
+    isolate->compilation_cache()->Clear();

[mvstanton, 2017/02/03 12:16:37]

Change to CollectAllAvailableGarbage().

+
     heap->CollectAllGarbage(i::Heap::kFinalizeIncrementalMarkingMask,
                             i::GarbageCollectionReason::kTesting);
     heap->CollectAllGarbage(i::Heap::kFinalizeIncrementalMarkingMask,
                             i::GarbageCollectionReason::kTesting);
 
     Object* raw_foo;
     {
       v8::HandleScope handle_scope(v8_isolate);
       v8::Local<v8::String> foo = v8_str("foo");
       CHECK(!foo.IsEmpty());
@@ skipping 88 matching lines @@
       v8::HandleScope handle_scope(v8_isolate);
       v8::Local<v8::Context>::New(v8_isolate, env)->Enter();
     }
     // Make sure all builtin scripts are cached.
     {
       HandleScope scope(isolate);
       for (int i = 0; i < Natives::GetBuiltinsCount(); i++) {
         isolate->bootstrapper()->SourceLookup<Natives>(i);
       }
     }
+
+    isolate->compilation_cache()->Clear();
+
     // If we don't do this then we end up with a stray root pointing at the
     // context even after we have disposed of env.
     heap->CollectAllGarbage(i::Heap::kFinalizeIncrementalMarkingMask,
                             i::GarbageCollectionReason::kTesting);
 
     {
       v8::HandleScope handle_scope(v8_isolate);
       v8::Local<v8::Context>::New(v8_isolate, env)->Exit();
     }
 
@@ skipping 618 matching lines @@
   CompileScript(isolate, src, src, &cache,
                 v8::ScriptCompiler::kProduceCodeCache);
 
   DisallowCompilation no_compile_expected(isolate);
   Handle<SharedFunctionInfo> copy = CompileScript(
       isolate, src, src, &cache, v8::ScriptCompiler::kConsumeCodeCache);
 
   CHECK(isolate->compilation_cache()
             ->LookupScript(src, src, 0, 0, v8::ScriptOriginOptions(),
                            isolate->native_context(), SLOPPY)
-            .ToHandleChecked()
-            .is_identical_to(copy));
+            .info() == *copy);
 
   delete cache;
 }
 
 TEST(CodeSerializerInternalizedString) {
   FLAG_serialize_toplevel = true;
   LocalContext context;
   Isolate* isolate = CcTest::i_isolate();
   isolate->compilation_cache()->Disable();  // Disable same-isolate code cache.
 
@@ skipping 1433 matching lines @@
   }
   delete[] blob.data;
 }
 
 TEST(SerializationMemoryStats) {
   FLAG_profile_deserialization = true;
   FLAG_always_opt = false;
   v8::StartupData blob = v8::V8::CreateSnapshotDataBlob();
   delete[] blob.data;
 }