Merge pull request #171 from serjflint/fix/zig/unsupported-exported_s…

…ymbols_list fix zig: unsupported exported_symbols_list
rust-cross · Sep 10, 2023 · fcb5d8a · fcb5d8a
2 parents fc2a082 + 83a4afc
commit fcb5d8a
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/zig.rs b/src/zig.rs
@@ -121,7 +121,7 @@ impl Zig {
                     // https://github.com/rust-lang/rust/blob/f0bc76ac41a0a832c9ee621e31aaf1f515d3d6a5/compiler/rustc_target/src/spec/windows_gnu_base.rs#L23
                     // https://github.com/rust-lang/rust/blob/2fb0e8d162a021f8a795fb603f5d8c0017855160/compiler/rustc_target/src/spec/windows_gnu_base.rs#L22
                     // https://github.com/rust-lang/rust/blob/f0bc76ac41a0a832c9ee621e31aaf1f515d3d6a5/compiler/rustc_target/src/spec/i686_pc_windows_gnu.rs#L16
-                    // zig doesn't support --disable-auto-image-base, --dynamicbase, and --large-address-aware
+                    // zig doesn't support --disable-auto-image-base, --dynamicbase and --large-address-aware
                     return None;
                 }
             } else if arg == "-Wl,--no-undefined-version" {
@@ -170,6 +170,13 @@ impl Zig {
                     }
                 }
             }
+            if is_macos {
+                if arg.starts_with("-Wl,--exported_symbols_list,") {
+                    // zig doesn't support --exported_symbols_list arg
+                    // https://clang.llvm.org/docs/ClangCommandLineReference.html#cmdoption-clang-exported_symbols_list
+                    return None;
+                }
+            }
             Some(arg.to_string())
         };
         let has_undefined_dynamic_lookup = |args: &[String]| {