AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Fix msan build

Browse Source 
This fixes an msan compile error after https://crrev.com/c/1587381.

TBR=jkummerow@chromium.org

No-Try: true
No-Tree-Checks: true
Bug: v8:9183
Change-Id: I50c5fc2ddb698a6441896ec393bce62eee7afdc9
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1587390
Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#61082}
This commit is contained in:
Clemens Hammacher 2019-04-29 14:59:40 +02:00 committed by Commit Bot
parent 4b0f9c856e
commit 99254554c1
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/snapshot/serializer-common.h
View File

@ -370,7 +370,7 @@ class Checksum {
#ifdef MEMORY_SANITIZER
// Computing the checksum includes padding bytes for objects like strings.
// Mark every object as initialized in the code serializer.
MSAN_MEMORY_IS_INITIALIZED(payload.start(), payload.length());
MSAN_MEMORY_IS_INITIALIZED(payload.begin(), payload.length());
#endif // MEMORY_SANITIZER
// Fletcher's checksum. Modified to reduce 64-bit sums to 32-bit.
uintptr_t a = 1;
@ -379,7 +379,7 @@ class Checksum {
// access payload through an uintptr_t pointer later on; and some
// architectures, e.g. arm, may generate instructions that expect correct
// alignment. However, we do not control alignment for external snapshots.
// DCHECK(IsAligned(reinterpret_cast<intptr_t>(payload.start()),
// DCHECK(IsAligned(reinterpret_cast<intptr_t>(payload.begin()),
// kIntptrSize));
DCHECK(IsAligned(payload.length(), kIntptrSize));
const uintptr_t* cur = reinterpret_cast<const uintptr_t*>(payload.begin());