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