test: Commonize shared configuration

Every platform needs the `__c_anonymous` config and most need to skip
`__uint128`, so just do these in a common place.

(backport <#4766>)
(cherry picked from commit c22bf18)

Author: tgross35

libc-test/build.rs

@@ -72,6 +72,16 @@ fn do_ctest() {
 fn ctest_cfg() -> ctest::TestGenerator {
     let mut cfg = ctest::TestGenerator::new();
     cfg.skip_private(true);
+
+    // Skip anonymous unions/structs.
+    cfg.skip_union(|u| u.ident().starts_with("__c_anonymous_"));
+    cfg.skip_struct(|s| s.ident().starts_with("__c_anonymous_"));
+    cfg.skip_alias(|ty| ty.ident().starts_with("__c_anonymous_"));
+
+    // __uint128 is not declared in C, but is an alias we export.
+    // FIXME(1.0): These aliases will eventually be removed.
+    cfg.skip_alias(|ty| ty.ident() == "__uint128");
+
     cfg
 }
 
@@ -307,10 +317,6 @@ fn test_apple(target: &str) {
         (x86_64, "crt_externs.h"),
     );
 
-    // Skip anonymous unions/structs.
-    cfg.skip_union(|u| u.ident().starts_with("__c_anonymous_"));
-    cfg.skip_struct(|s| s.ident().starts_with("__c_anonymous_"));
-
     cfg.skip_struct(|s| {
         match s.ident() {
             // FIXME(union): actually a union
@@ -324,15 +330,6 @@ fn test_apple(target: &str) {
         }
     });
 
-    cfg.skip_alias(|ty| ty.ident().starts_with("__c_anonymous_"));
-    cfg.skip_alias(|ty| {
-        match ty.ident() {
-            // FIXME(macos): "'__uint128' undeclared" in C
-            "__uint128" => true,
-            _ => false,
-        }
-    });
-
     cfg.skip_const(move |constant| {
         match constant.ident() {
             // They're declared via `deprecated_mach` and we don't support it anymore.
@@ -590,9 +587,6 @@ fn test_openbsd(target: &str) {
         }
     });
 
-    // Skip anonymous unions/structs.
-    cfg.skip_union(|u| u.ident().starts_with("__c_anonymous_"));
-    cfg.skip_struct(|s| s.ident().starts_with("__c_anonymous_"));
     cfg.skip_struct(move |ty| {
         match ty.ident() {
             // FIXME(union): actually a union
@@ -729,14 +723,6 @@ fn test_cygwin(target: &str) {
         _ => false,
     });
 
-    cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
-
-        false
-    });
-
     cfg.rename_struct_field(move |struct_, field| {
         match field.ident() {
             // Our stat *_nsec fields normally don't actually exist but are part
@@ -869,17 +855,12 @@ fn test_windows(target: &str) {
     });
 
     cfg.skip_struct(move |struct_| {
-        let ty = struct_.ident();
-        if ty.starts_with("__c_anonymous_") {
-            return true;
-        }
-        match ty {
+        match struct_.ident() {
             // FIXME(windows): The size and alignment of this struct are incorrect
             "timespec" if gnu && i686 => true,
             _ => false,
         }
     });
-    cfg.skip_union(move |union_| union_.ident().starts_with("__c_anonymous_"));
 
     cfg.skip_const(move |constant| {
         match constant.ident() {
@@ -1141,15 +1122,12 @@ fn test_solarish(target: &str) {
 
     cfg.skip_union(|union_| {
         // the union handling is a mess
-        if union_.ident().starts_with("__c_anonymous_") || union_.ident().contains("door_desc_t_") {
+        if union_.ident().contains("door_desc_t_") {
             return true;
         }
         false
     });
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         // the union handling is a mess
         if struct_.ident().contains("door_desc_t_") {
             return true;
@@ -1392,13 +1370,7 @@ fn test_netbsd(target: &str) {
         }
     });
 
-    cfg.skip_struct(|ty| ty.ident().starts_with("__c_anonymous_"));
-    cfg.skip_union(|ty| ty.ident().starts_with("__c_anonymous_"));
-
     cfg.skip_alias(move |ty| {
-        if ty.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match ty.ident() {
             // FIXME(netbsd): sighandler_t is crazy across platforms
             "sighandler_t" => true,
@@ -1627,9 +1599,6 @@ fn test_dragonflybsd(target: &str) {
     });
 
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match struct_.ident() {
             // FIXME(dragonflybsd): These are tested as part of the linux_fcntl tests since
             // there are header conflicts when including them with all the other
@@ -2007,21 +1976,14 @@ fn test_android(target: &str) {
             "posix_spawn_file_actions_t" => true,
             "posix_spawnattr_t" => true,
 
-            // FIXME(android): "'__uint128' undeclared" in C
-            "__uint128" => true,
             // Added in API level 24
             "if_nameindex" => true,
 
             _ => false,
         }
     });
 
-    cfg.skip_union(move |union_| union_.ident().starts_with("__c_anonymous_"));
-
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match struct_.ident() {
             // These are tested as part of the linux_fcntl tests since there are
             // header conflicts when including them with all the other structs.
@@ -2812,13 +2774,8 @@ fn test_freebsd(target: &str) {
         }
     });
 
-    cfg.skip_union(|u| u.ident().starts_with("__c_anonymous_"));
     cfg.skip_struct(move |struct_| {
-        let ty = struct_.ident();
-        if ty.starts_with("__c_anonymous_") {
-            return true;
-        }
-        match ty {
+        match struct_.ident() {
             // `procstat` is a private struct
             "procstat" => true,
 
@@ -3132,10 +3089,6 @@ fn test_emscripten(target: &str) {
     });
 
     cfg.skip_union(|union_| {
-        if union_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
-
         match union_.ident() {
             // No epoll support
             // https://github.com/emscripten-core/emscripten/issues/5033
@@ -3154,9 +3107,6 @@ fn test_emscripten(target: &str) {
     });
 
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match struct_.ident() {
             // This is actually a union, not a struct
             "sigval" => true,
@@ -3430,17 +3380,11 @@ fn test_neutrino(target: &str) {
             // Does not exist in Neutrino
             "locale_t" => true,
 
-            // FIXME: "'__uint128' undeclared" in C
-            "__uint128" => true,
-
             _ => false,
         }
     });
 
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match struct_.ident() {
             "Elf64_Phdr" | "Elf32_Phdr" => true,
 
@@ -4014,9 +3958,6 @@ fn test_linux(target: &str) {
             // specific type.
             "Ioctl" => true,
 
-            // FIXME: "'__uint128' undeclared" in C
-            "__uint128" => true,
-
             t => {
                 if musl {
                     // LFS64 types have been removed in musl 1.2.4+
@@ -4028,10 +3969,6 @@ fn test_linux(target: &str) {
         }
     });
 
-    // Skip structs and enums that are unnamed in C.
-    cfg.skip_union(move |union_| union_.ident().starts_with("__c_anonymous_"));
-    cfg.skip_struct(move |struct_| struct_.ident().starts_with("__c_anonymous_"));
-
     cfg.skip_struct(move |struct_| {
         let ty = struct_.ident();
 
@@ -4113,10 +4050,7 @@ fn test_linux(target: &str) {
             "xdp_umem_reg" => true,
 
             // FIXME(linux): Requires >= 6.8 kernel headers.
-            "xsk_tx_metadata"
-            | "__c_anonymous_xsk_tx_metadata_union"
-            | "xsk_tx_metadata_request"
-            | "xsk_tx_metadata_completion" => true,
+            "xsk_tx_metadata" | "xsk_tx_metadata_request" | "xsk_tx_metadata_completion" => true,
 
             // A new field was added in kernel 5.4, this is the old version for backwards compatibility.
             // https://github.com/torvalds/linux/commit/77cd0d7b3f257fd0e3096b4fdcff1a7d38e99e10
@@ -4797,9 +4731,9 @@ fn test_linux_like_apis(target: &str) {
     if linux || android || emscripten {
         // test strerror_r from the `string.h` header
         config_gnu_bits(target, &mut cfg);
-        headers!(cfg, "string.h",);
 
-        cfg.skip_alias(|_| true)
+        cfg.header("string.h")
+            .skip_alias(|_| true)
             .skip_static(|_| true)
             .skip_const(|_| true)
             .skip_struct(|_| true)
@@ -5098,11 +5032,7 @@ fn test_haiku(target: &str) {
         "support/TypeConstants.h",
     );
 
-    cfg.skip_union(|union_| union_.ident().starts_with("__c_anonymous_"));
     cfg.skip_struct(move |struct_| {
-        if struct_.ident().starts_with("__c_anonymous_") {
-            return true;
-        }
         match struct_.ident() {
             // FIXME(union): actually a union
             "sigval" => true,