| ... | @@ -2370,17 +2370,6 @@ pub fn addModuleTests(b: *std.Build, options: ModuleTestOptions) *Step { | ... | @@ -2370,17 +2370,6 @@ pub fn addModuleTests(b: *std.Build, options: ModuleTestOptions) *Step { |
| 2370 | const resolved_target = b.resolveTargetQuery(test_target.target); | 2370 | const resolved_target = b.resolveTargetQuery(test_target.target); |
| 2371 | const target = &resolved_target.result; | 2371 | const target = &resolved_target.result; |
| 2372 | | 2372 | |
| 2373 | if (test_target.link_libc == false and target.requiresLibC()) continue; | | |
| 2374 | // If the target requires libc, there's no point building the cases that | | |
| 2375 | // don't explicitly link libc as they'll just end up actually linking | | |
| 2376 | // libc anyway, thus creating duplicate work and making -Dskip-libc not | | |
| 2377 | // work as expected. | | |
| 2378 | if (test_target.link_libc == null and target.requiresLibC()) continue; | | |
| 2379 | // These targets don't strictly require libc, but we don't yet have a | | |
| 2380 | // syscall layer for them, so the compiler links libc by default. They | | |
| 2381 | // therefore get the same treatment here. | | |
| 2382 | if (test_target.link_libc == null and (target.os.tag == .freebsd or target.os.tag == .netbsd or target.os.tag == .openbsd)) continue; | | |
| 2383 | | | |
| 2384 | if (!options.test_extra_targets and test_target.extra_target) continue; | 2373 | if (!options.test_extra_targets and test_target.extra_target) continue; |
| 2385 | | 2374 | |
| 2386 | if (options.skip_non_native and !test_target.target.isNative()) | 2375 | if (options.skip_non_native and !test_target.target.isNative()) |