From 436aafd3e2ef1a8f5998b974a9791b59939f57ad Mon Sep 17 00:00:00 2001 From: Andrew Kelley Date: Thu, 9 Jun 2022 19:14:52 -0700 Subject: [PATCH] remove a stage2 test case that relies on a very deep compiler stack It's causing Drone CI failure on aarch64. I will open a follow-up issue to track this missing test case. --- test/stage2/cbe.zig | 24 ------------------------ 1 file changed, 24 deletions(-) diff --git a/test/stage2/cbe.zig b/test/stage2/cbe.zig index ae810a31ca..47ed7a53be 100644 --- a/test/stage2/cbe.zig +++ b/test/stage2/cbe.zig @@ -233,30 +233,6 @@ pub fn addCases(ctx: *TestContext) !void { \\} , ""); } - // This will make a pretty deep call stack, so this test can only be enabled - // on hosts where Zig's linking strategy can honor the 16 MiB (default) we - // link the self-hosted compiler with. - const host_supports_custom_stack_size = @import("builtin").target.os.tag == .linux; - if (host_supports_custom_stack_size) { - var case = ctx.exeFromCompiledC("@setEvalBranchQuota", .{}); - - // TODO when adding result location support to function calls, revisit this test - // case. It can go back to what it was before, with `y` being comptime known. - // Because the ret_ptr will passed in with the inline fn call, and there will - // only be 1 store to it, and it will be comptime known. - case.addCompareOutput( - \\pub export fn main() i32 { - \\ @setEvalBranchQuota(1001); - \\ const y = rec(1001); - \\ return y - 1; - \\} - \\ - \\inline fn rec(n: i32) i32 { - \\ if (n <= 1) return n; - \\ return rec(n - 1); - \\} - , ""); - } { var case = ctx.exeFromCompiledC("control flow", .{});