Commit b8745f68 authored by earthdok's avatar earthdok Committed by Commit bot

Revert r257779: "MemorySanitizer: adjust stack size to work around an MSan bug."

The bug has long been fixed so can remove this workaround.

BUG=353687
TBR=thakis@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/536903003

Cr-Commit-Position: refs/heads/master@{#293145}
parent bb113630
...@@ -107,12 +107,11 @@ void InitOnThread() {} ...@@ -107,12 +107,11 @@ void InitOnThread() {}
void TerminateOnThread() {} void TerminateOnThread() {}
size_t GetDefaultThreadStackSize(const pthread_attr_t& attributes) { size_t GetDefaultThreadStackSize(const pthread_attr_t& attributes) {
#if !defined(THREAD_SANITIZER) && !defined(MEMORY_SANITIZER) #if !defined(THREAD_SANITIZER)
return 0; return 0;
#else #else
// ThreadSanitizer bloats the stack heavily. Evidence has been that the // ThreadSanitizer bloats the stack heavily. Evidence has been that the
// default stack size isn't enough for some browser tests. // default stack size isn't enough for some browser tests.
// MemorySanitizer needs this as a temporary fix for http://crbug.com/353687
return 2 * (1 << 23); // 2 times 8192K (the default stack size on Linux). return 2 * (1 << 23); // 2 times 8192K (the default stack size on Linux).
#endif #endif
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment