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

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

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

=======================================
--- /branches/3.3/src/heap-inl.h        Wed May  4 05:43:48 2011
+++ /branches/3.3/src/heap-inl.h        Thu Jul 21 05:28:18 2011
@@ -368,11 +368,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;
@@ -390,8 +386,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.3/src/version.cc        Wed Jul 13 07:25:10 2011
+++ /branches/3.3/src/version.cc        Thu Jul 21 05:28:18 2011
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     3
 #define BUILD_NUMBER      10
-#define PATCH_LEVEL       21
+#define PATCH_LEVEL       22
 // 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