From 6c46f4f11cdd56fcd12c86d121259c738b7a8376 Mon Sep 17 00:00:00 2001 From: Josh Stone Date: Fri, 4 Aug 2017 00:13:11 -0700 Subject: [PATCH] Use LLVM_LINK_LLVM_DYLIB only on linux-gnu and apple-darwin --- src/bootstrap/native.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/bootstrap/native.rs b/src/bootstrap/native.rs index ce0052a5fb6..ee0eca5d482 100644 --- a/src/bootstrap/native.rs +++ b/src/bootstrap/native.rs @@ -133,8 +133,8 @@ impl Step for Llvm { // This setting makes the LLVM tools link to the dynamic LLVM library, // which saves both memory during parallel links and overall disk space // for the tools. We don't distribute any of those tools, so this is - // just a local concern. However, this doesn't seem to work on Windows. - if !target.contains("windows") { + // just a local concern. However, it doesn't work well everywhere. + if target.contains("linux-gnu") || target.contains("apple-darwin") { cfg.define("LLVM_LINK_LLVM_DYLIB", "ON"); }