Record code slots that may point to evacuation candidate objects after deoptimizing them.

src/deoptimizer.cc

 // 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.
 
 #include "src/v8.h"
 
 #include "src/accessors.h"
 #include "src/codegen.h"
 #include "src/cpu-profiler.h"
 #include "src/deoptimizer.h"
@@ skipping 399 matching lines @@
         SharedFunctionInfo::cast(deopt_data->SharedFunctionInfo());
     shared->EvictFromOptimizedCodeMap(codes[i], "deoptimized code");
 
     // Do platform-specific patching to force any activations to lazy deopt.
     PatchCodeForDeoptimization(isolate, codes[i]);
 
     // We might be in the middle of incremental marking with compaction.
     // Ignore all slots that might have been recorded on the deoptimized code
     // object.
     isolate->heap()->mark_compact_collector()->RemoveObjectSlots(codes[i]);
+
+    // After removing all recorded slots of the code object, we add the slots
+    // again that point to objects on evacuation candidates.

[Michael Starzinger, 2015/07/03 10:46:09]

nit: Let's add to the comment that this is only about slots in the header of the
code object, not the instruction stream.

[Hannes Payer (out of office), 2015/07/03 10:55:53]

Done.

+    isolate->heap()->mark_compact_collector()->RecordDeoptimizedCodeSlots(
+        codes[i]);
   }
 }
 
 
 void Deoptimizer::DeoptimizeAll(Isolate* isolate) {
   if (FLAG_trace_deopt) {
     CodeTracer::Scope scope(isolate->GetCodeTracer());
     PrintF(scope.file(), "[deoptimize all code in all contexts]\n");
   }
   DisallowHeapAllocation no_allocation;
@@ skipping 2941 matching lines @@
       DCHECK(value_info->IsMaterializedObject());
 
       value_info->value_ =
           Handle<Object>(previously_materialized_objects->get(i), isolate_);
     }
   }
 }
 
 }  // namespace internal
 }  // namespace v8