Revision: 8710
Author:   [email protected]
Date:     Thu Jul 21 05:28:10 2011
Log: Merge fix for incorrect overlap check in mark compact collector to 3.2
branch.

Bleeding edge revision 8704.
Review URL: http://codereview.chromium.org/7464023
http://code.google.com/p/v8/source/detail?r=8710

Modified:
 /branches/3.2/src/heap-inl.h
 /branches/3.2/src/version.cc

=======================================
--- /branches/3.2/src/heap-inl.h        Wed Mar 23 04:19:56 2011
+++ /branches/3.2/src/heap-inl.h        Thu Jul 21 05:28:10 2011
@@ -367,11 +367,7 @@

   int size_in_words = byte_size / kPointerSize;

-  if ((dst < src) || (dst >= (src + size_in_words))) {
-    ASSERT((dst >= (src + size_in_words)) ||
-           ((OffsetFrom(reinterpret_cast<Address>(src)) -
-             OffsetFrom(reinterpret_cast<Address>(dst))) >= kPointerSize));
-
+  if ((dst < src) || (dst >= (src + byte_size))) {
     Object** src_slot = reinterpret_cast<Object**>(src);
     Object** dst_slot = reinterpret_cast<Object**>(dst);
     Object** end_slot = src_slot + size_in_words;
@@ -389,8 +385,7 @@
                                                    Address src,
                                                    int byte_size) {
   ASSERT(IsAligned(byte_size, kPointerSize));
-  ASSERT((dst >= (src + byte_size)) ||
-         ((OffsetFrom(src) - OffsetFrom(dst)) >= kPointerSize));
+  ASSERT((dst < src) || (dst >= (src + byte_size)));

   CopyBlockToOldSpaceAndUpdateRegionMarks(dst, src, byte_size);
 }
=======================================
--- /branches/3.2/src/version.cc        Thu Jul 14 03:40:50 2011
+++ /branches/3.2/src/version.cc        Thu Jul 21 05:28:10 2011
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     2
 #define BUILD_NUMBER      10
-#define PATCH_LEVEL       30
+#define PATCH_LEVEL       31
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0

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

Reply via email to