Skip to content

Commit

Permalink
Increase stack size for CompileDeepTree_NoStackOverflowFast
Browse files Browse the repository at this point in the history
Should allow it to run with checked coreclr builds, but still
test the stack-overflow guards without running for a long time.

Fixes #17550
  • Loading branch information
JonHanna committed Mar 29, 2017
1 parent a44c930 commit dd3c6be
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/System.Linq.Expressions/tests/CompilerTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@ public static void CompileDeepTree_NoStackOverflowFast(bool useInterpreter)
e = Expression.Add(e, Expression.Constant(1));

Func<int> f = null;
// Request a stack size of 64K to get very small stack.
// Request a stack size of 192K to get very small stack.
// This reduces the size of tree needed to risk a stack overflow.
// This though will only risk overflow once, so the outerloop test
// above is still needed.
Thread t = new Thread(() => f = Expression.Lambda<Func<int>>(e).Compile(useInterpreter), 65536);
Thread t = new Thread(() => f = Expression.Lambda<Func<int>>(e).Compile(useInterpreter), 128 * 1024);
t.Start();
t.Join();

Expand Down

0 comments on commit dd3c6be

Please sign in to comment.