[PATCH] D56898: gn build: unbreak mac (and maybe win) after r351258, r351277

Phabricator via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 17 20:15:46 PST 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL351519: gn build: unbreak mac (and maybe win) after r351258, r351277 (authored by nico, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D56898?vs=182448&id=182452#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56898/new/

https://reviews.llvm.org/D56898

Files:
  llvm/trunk/utils/gn/build/toolchain/BUILD.gn
  llvm/trunk/utils/gn/secondary/BUILD.gn


Index: llvm/trunk/utils/gn/secondary/BUILD.gn
===================================================================
--- llvm/trunk/utils/gn/secondary/BUILD.gn
+++ llvm/trunk/utils/gn/secondary/BUILD.gn
@@ -5,10 +5,12 @@
 group("default") {
   deps = [
     "//clang/test",
-    "//compiler-rt/test/hwasan",
     "//lld/test",
     "//llvm/test",
   ]
+  if (current_os == "linux" || current_os == "android") {
+    deps += [ "//compiler-rt/test/hwasan" ]
+  }
 
   testonly = true
 }
Index: llvm/trunk/utils/gn/build/toolchain/BUILD.gn
===================================================================
--- llvm/trunk/utils/gn/build/toolchain/BUILD.gn
+++ llvm/trunk/utils/gn/build/toolchain/BUILD.gn
@@ -157,13 +157,17 @@
     cc = "bin/clang"
     cxx = "bin/clang++"
     ld = cxx
-    ar = "bin/llvm-ar"
+    if (current_os != "mac") {
+      ar = "bin/llvm-ar"
+    }
 
     deps = [
       "//:clang($host_toolchain)",
       "//:lld($host_toolchain)",
-      "//:llvm-ar($host_toolchain)",
     ]
+    if (current_os != "mac") {
+      deps += [ "//:llvm-ar($host_toolchain)" ]
+    }
   }
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56898.182452.patch
Type: text/x-patch
Size: 1101 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190118/2094401e/attachment.bin>


More information about the llvm-commits mailing list