Commit ec9d37a5 authored by Robin Cheng's avatar Robin Cheng Committed by Facebook GitHub Bot

Adjust stack limits for StackTraceSizeLimitTest for TSAN.

Summary: TSAN just needs more stack doing this...

Reviewed By: yfeldblum

Differential Revision: D23484864

fbshipit-source-id: e5412982349f73f8ead795403dfbde2830f4e5fd
parent 8b53f907
...@@ -74,30 +74,30 @@ TEST(StackTraceSizeLimitTest, FiberLimit) { ...@@ -74,30 +74,30 @@ TEST(StackTraceSizeLimitTest, FiberLimit) {
// Initial run // Initial run
t(fBaseline, 10000); t(fBaseline, 10000);
#ifdef NDEBUG // The amount of stack needed varies based on compilation modes.
// Baseline if (folly::kIsDebug) {
t(fBaseline, 1600); if (folly::kIsSanitizeAddress) {
// Standard version t(fBaseline, 0);
t(fStack, 6800); t(fStack, 0);
// Heap version t(fHeap, 0);
t(fHeap, 2200); } else if (folly::kIsSanitizeThread) {
#else t(fBaseline, 3700);
// Values for opt builds t(fStack, 9000);
t(fHeap, 5000);
#ifndef FOLLY_SANITIZE_ADDRESS } else {
// Baseline t(fBaseline, 3700);
t(fBaseline, 3700); t(fStack, 8000);
// Standard version t(fHeap, 3700);
t(fStack, 8000); }
// Heap version } else {
t(fHeap, 3700); if (folly::kIsSanitizeThread) {
#else t(fBaseline, 2500);
// Baseline t(fStack, 6800);
t(fBaseline, 0); t(fHeap, 3500);
// Standard version } else {
t(fStack, 0); t(fBaseline, 1600);
// Heap version t(fStack, 6800);
t(fHeap, 0); t(fHeap, 2200);
#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