Reviewers: Jakob,

Description:
Ensure code flushing is disabled in incremental marking.

This fixes some test failures since r12714 which shared the code for
code flushing between full and incremental marking and is only a quick
fix until we can enable incremental code flushing.

[email protected]
BUG=v8:1609


Please review this at https://codereview.chromium.org/11137014/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/mark-compact.cc


Index: src/mark-compact.cc
diff --git a/src/mark-compact.cc b/src/mark-compact.cc
index f8fd0cdeec59fb18cfce7aab46381a281749688c..37fec4e8b8312e2adf9b38113aefa257aa28cbf4 100644
--- a/src/mark-compact.cc
+++ b/src/mark-compact.cc
@@ -2003,6 +2003,9 @@ void MarkCompactCollector::AfterMarking() {
   // Flush code from collected candidates.
   if (is_code_flushing_enabled()) {
     code_flusher_->ProcessCandidates();
+ // TODO(1609) Currently incremental marker does not support code flushing, + // we need to disable it before incremental marking steps for next cycle.
+    EnableCodeFlushing(false);
   }

   if (!FLAG_watch_ic_patching) {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to