[Interpreter] Make ignition compiler eagerly.

test/cctest/test-heap-profiler.cc

 // Copyright 2011 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 349 matching lines @@
     const v8::HeapGraphEdge* prop = lazy_code->GetChild(i);
     const v8::HeapGraphNode* node = prop->GetToNode();
     if (node->GetType() == v8::HeapGraphNode::kArray) {
       if (HasString(node, "x")) {
         lazy_references_x = true;
         break;
       }
     }
   }
   CHECK(compiled_references_x);
-  CHECK(!lazy_references_x);
+  if (i::FLAG_lazy && !i::FLAG_ignition) {

[Michael Starzinger, 2016/03/24 13:27:14]

nit: Can we also check for "FLAG_ignition_eager" here? Or am I missing something
in this case?

[rmcilroy, 2016/03/24 13:48:25]

Missed this, good catch, thanks. Done

+    CHECK(!lazy_references_x);
+  }
 }
 
 
 TEST(HeapSnapshotHeapNumbers) {
   LocalContext env;
   v8::HandleScope scope(env->GetIsolate());
   v8::HeapProfiler* heap_profiler = env->GetIsolate()->GetHeapProfiler();
   CompileRun(
       "a = 1;    // a is Smi\n"
       "b = 2.5;  // b is HeapNumber");
@@ skipping 2623 matching lines @@
   v8::base::SmartPointer<v8::AllocationProfile> profile(
       heap_profiler->GetAllocationProfile());
   CHECK(!profile.is_empty());
   const char* names[] = {"(V8 API)"};
   auto node = FindAllocationProfileNode(
       *profile, Vector<const char*>(names, arraysize(names)));
   CHECK(node);
 
   heap_profiler->StopSamplingHeapProfiler();
 }