Commit e3dd4dc91d65702c085dac9cc4d85aebfa308dfa | zig

author	Andrew Kelley <andrew@ziglang.org> 2021-05-11 20:28:14-04:00
committer	GitHub <noreply@github.com> 2021-05-11 20:28:14-04:00
log	e3dd4dc91d65702c085dac9cc4d85aebfa308dfa
tree	ee8aadf75cd14cd5bf9833b368b2553aae2365a8
parent	e6881d4373ce355f3f4565c22c125870ed8fcfc8
parent	f8cf106fc971c9c61709443860b1f728ca4cfc9a
signature	Signed by PGP key 4AEE18F83AFDEB23

Merge pull request #8737 from ifreund/link-system-libc

stage2: use system libc when targeting the native OS/ABI

4 files changed, 26 insertions(+), 19 deletions(-)

ci
- azure
  - macos_arm64_script
src
- Compilation.zig
- glibc.zig
- link
  - Elf.zig

ci/azure/macos_arm64_script+4-4

 @@ -55,11 +55,11 @@ cmake .. \
   -DZIG_TARGET_MCPU="$HOST_MCPU" \
   -DZIG_STATIC=ON
-make $JOBS install
 unset CC
 unset CXX
+make $JOBS install
 # Build zig compiler cross-compiled for arm64
 cd $ZIGDIR
 @@ -79,11 +79,11 @@ cmake .. \
   -DZIG_EXECUTABLE="$ZIG" \
   -DZIG_STATIC=ON
-make $JOBS install
 unset CC
 unset CXX
+make $JOBS install
 if [ "${BUILD_REASON}" != "PullRequest" ]; then
   mv ../LICENSE release/

src/Compilation.zig+8-8

 @@ -848,7 +848,7 @@ pub fn create(gpa: *Allocator, options: InitOptions) !*Compilation {
             arena,
             options.zig_lib_directory.path.?,
             options.target,
-            options.is_native_os,
+            options.is_native_abi,
             link_libc,
             options.libc_installation,
         );
 @@ -2885,7 +2885,7 @@ fn detectLibCIncludeDirs(
     arena: *Allocator,
     zig_lib_dir: []const u8,
     target: Target,
-    is_native_os: bool,
+    is_native_abi: bool,
     link_libc: bool,
     libc_installation: ?*const LibCInstallation,
 ) !LibCDirs {
 @@ -2900,6 +2900,12 @@ fn detectLibCIncludeDirs(
         return detectLibCFromLibCInstallation(arena, target, lci);
     }
+    if (is_native_abi) {
+        const libc = try arena.create(LibCInstallation);
+        libc.* = try LibCInstallation.findNative(.{ .allocator = arena });
+        return detectLibCFromLibCInstallation(arena, target, libc);
+    }
     if (target_util.canBuildLibC(target)) {
         const generic_name = target_util.libCGenericName(target);
         // Some architectures are handled by the same set of headers.
 @@ -2950,12 +2956,6 @@ fn detectLibCIncludeDirs(
         };
     }
-    if (is_native_os) {
-        const libc = try arena.create(LibCInstallation);
-        libc.* = try LibCInstallation.findNative(.{ .allocator = arena });
-        return detectLibCFromLibCInstallation(arena, target, libc);
-    }
     return LibCDirs{
         .libc_include_dir_list = &[0][]u8{},
         .libc_installation = null,

src/glibc.zig+5-4

 @@ -764,16 +764,17 @@ pub fn buildSharedObjects(comp: *Compilation) !void {
                 .lt => continue,
                 .gt => {
                     // TODO Expose via compile error mechanism instead of log.
-                    std.log.warn("invalid target glibc version: {}", .{target_version});
+                    std.log.err("invalid target glibc version: {}", .{target_version});
                     return error.InvalidTargetGLibCVersion;
                 },
             }
-        } else blk: {
+        } else {
             const latest_index = metadata.all_versions.len - 1;
-            std.log.warn("zig cannot build new glibc version {}; providing instead {}", .{
+            // TODO Expose via compile error mechanism instead of log.
+            std.log.err("zig does not yet provide glibc version {}, the max provided version is {}", .{
                 target_version, metadata.all_versions[latest_index],
             });
-            break :blk latest_index;
+            return error.InvalidTargetGLibCVersion;
         };
         {
             var map_contents = std.ArrayList(u8).init(arena);

src/link/Elf.zig+9-3

 @@ -1650,9 +1650,15 @@ fn linkWithLLD(self: *Elf, comp: *Compilation) !void {
                 if (self.base.options.libc_installation != null) {
                     const needs_grouping = self.base.options.link_mode == .Static;
                     if (needs_grouping) try argv.append("--start-group");
-                    try argv.append("-lm");
+                    // This matches the order of glibc.libs
-                    try argv.append("-lpthread");
+                    try argv.appendSlice(&[_][]const u8{
-                    try argv.append("-lc");
+                        "-lm",
+                        "-lpthread",
+                        "-lc",
+                        "-ldl",
+                        "-lrt",
+                        "-lutil",
+                    });
                     if (needs_grouping) try argv.append("--end-group");
                 } else if (target.isGnuLibC()) {
                     try argv.append(comp.libunwind_static_lib.?.full_object_path);