Annotate ThreadData::PushToHeadOfList for MemorySanitizer.

base/compiler_specific.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_COMPILER_SPECIFIC_H_
 #define BASE_COMPILER_SPECIFIC_H_
 
 #include "build/build_config.h"
 
 #if defined(COMPILER_MSVC)
@@ skipping 148 matching lines @@
 #define PRINTF_FORMAT(format_param, dots_param)
 #endif
 
 // WPRINTF_FORMAT is the same, but for wide format strings.
 // This doesn't appear to yet be implemented in any compiler.
 // See http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38308 .
 #define WPRINTF_FORMAT(format_param, dots_param)
 // If available, it would look like:
 //   __attribute__((format(wprintf, format_param, dots_param)))
 
+
+// MemorySanitizer annotations.
+#ifdef MEMORY_SANITIZER
+extern "C" {
+  void __msan_unpoison(const void *p, unsigned long s);

[Mark Mentovai, 2013/03/20 13:34:58]

No indentation within extern "C". We treat it the same as a namespace.

[eugenis, 2013/03/20 13:40:29]

Done.

+}

[Mark Mentovai, 2013/03/20 13:34:58]

}  // extern "C"

(again, like a namespace)

[eugenis, 2013/03/20 13:40:29]

Done.

+
+// Mark a memory region fully initialized.
+// Use this to annotate the code that deliberately reads uninitialized data,

[Mark Mentovai, 2013/03/20 13:34:58]

Remove the word “the.”

[eugenis, 2013/03/20 13:40:29]

Done.

+// for example a GC scavenging root set pointers from stack.

[Mark Mentovai, 2013/03/20 13:34:58]

And here, “stack” needs “the.”

[eugenis, 2013/03/20 13:40:29]

Done.

+#define MSAN_UNPOISON(p, s)  __msan_unpoison(p, s)
+#else // MEMORY_SANITIZER

[Mark Mentovai, 2013/03/20 13:34:58]

At least two spaces between the last non-whitespace thing on a line and the
opening // of a line comment. Same on line 182.

[eugenis, 2013/03/20 13:40:29]

Done.

+#define MSAN_UNPOISON(p, s)
+#endif // MEMORY_SANITIZER
+
 #endif  // BASE_COMPILER_SPECIFIC_H_