Commit e437efd6015cdc369a2ca632565d105e9f04f20f | zig

author	Alex Rønne Petersen <alex@alexrp.com> 2026-01-23 19:47:12+01:00
committer	Alex Rønne Petersen <alex@alexrp.com> 2026-01-23 21:16:03+01:00
log	e437efd6015cdc369a2ca632565d105e9f04f20f
tree	9570a204defe1aecfcc5e4cb0f2ef4d6cc5e6af4
parent	c699bb81347dc0c9371c7c6c2cddab457cbe02ba
signature	Signed by SSH key SHA256:7B/LJ7bpR1eX8aCXSr4mtd5M45VMPKcx9zY8e95b5QM

test: enable thumb-windows-gnu module tests

We use long calls for these just like thumb*-linux-* to prevent range issues as
the binaries grow larger over time.

We also need function and data sections due to the many __stack_chk_guard
references within the std test binary; without these options, the linker is not
able to insert range thunks in between functions because the std binary just has
one giant .text section that's opaque to the linker.

closes https://codeberg.org/ziglang/zig/issues/30923

1 files changed, 38 insertions(+), 27 deletions(-)

test
- tests.zig

test/tests.zig+38-27

 @@ -28,6 +28,8 @@ const TestTarget = struct {
     use_lld: ?bool = null,
     pic: ?bool = null,
     strip: ?bool = null,
+    function_sections: ?bool = null,
+    data_sections: ?bool = null,
     skip_modules: []const []const u8 = &.{},
     // This is intended for targets that, for any reason, shouldn't be run as part of a normal test
 @@ -40,7 +42,7 @@ const test_targets = blk: {
     // getBaselineCpuFeatures calls populateDependencies which has a O(N ^ 2) algorithm
     // (where N is roughly 160, which technically makes it O(1), but it adds up to a
     // lot of branches)
-    @setEvalBranchQuota(60000);
+    @setEvalBranchQuota(80_000);
     break :blk [_]TestTarget{
         // Native Targets
 @@ -1526,36 +1528,43 @@ const test_targets = blk: {
         },
         .{
-            .target = .{
+            .target = std.Target.Query.parse(.{
-                .cpu_arch = .thumb,
+                .arch_os_abi = "thumb-windows-msvc",
-                .os_tag = .windows,
+                .cpu_features = "baseline+long_calls",
-                .abi = .msvc,
+            }) catch unreachable,
-            },
+            .pic = false, // Long calls don't work with PIC.
+            .function_sections = true,
+            .data_sections = true,
         },
         .{
-            .target = .{
+            .target = std.Target.Query.parse(.{
-                .cpu_arch = .thumb,
+                .arch_os_abi = "thumb-windows-msvc",
-                .os_tag = .windows,
+                .cpu_features = "baseline+long_calls",
-                .abi = .msvc,
+            }) catch unreachable,
-            },
             .link_libc = true,
+            .pic = false, // Long calls don't work with PIC.
+            .function_sections = true,
+            .data_sections = true,
+        },
+        .{
+            .target = std.Target.Query.parse(.{
+                .arch_os_abi = "thumb-windows-gnu",
+                .cpu_features = "baseline+long_calls",
+            }) catch unreachable,
+            .pic = false, // Long calls don't work with PIC.
+            .function_sections = true,
+            .data_sections = true,
+        },
+        .{
+            .target = std.Target.Query.parse(.{
+                .arch_os_abi = "thumb-windows-gnu",
+                .cpu_features = "baseline+long_calls",
+            }) catch unreachable,
+            .link_libc = true,
+            .pic = false, // Long calls don't work with PIC.
+            .function_sections = true,
+            .data_sections = true,
         },
-        // https://github.com/ziglang/zig/issues/24016
-        // .{
-        //     .target = .{
-        //         .cpu_arch = .thumb,
-        //         .os_tag = .windows,
-        //         .abi = .gnu,
-        //     },
-        // },
-        // .{
-        //     .target = .{
-        //         .cpu_arch = .thumb,
-        //         .os_tag = .windows,
-        //         .abi = .gnu,
-        //     },
-        //     .link_libc = true,
-        // },
         .{
             .target = .{
 @@ -2454,6 +2463,8 @@ fn addOneModuleTest(
     if (options.build_options) |build_options| {
         these_tests.root_module.addOptions("build_options", build_options);
     }
+    if (test_target.function_sections) |fs| these_tests.link_function_sections = fs;
+    if (test_target.data_sections) |ds| these_tests.link_data_sections = ds;
     const single_threaded_suffix = if (test_target.single_threaded == true) "-single" else "";
     const backend_suffix = if (test_target.use_llvm == true)
         "-llvm"