[PATCH] Fixed llvm-build when no targets are enabled.

Greg Fitzgerald garious at gmail.com
Fri Apr 18 10:06:30 PDT 2014


Hi ddunbar,

llvm-build fails when -DLLVM_TARGETS_TO_BUILD="".  This patch fixes it.

http://reviews.llvm.org/D3428

Files:
  utils/llvm-build/llvmbuild/main.py

Index: utils/llvm-build/llvmbuild/main.py
===================================================================
--- utils/llvm-build/llvmbuild/main.py
+++ utils/llvm-build/llvmbuild/main.py
@@ -719,7 +719,9 @@
         enable_targets = available_targets.values()
     else:
         # We support both space separated and semi-colon separated lists.
-        if ' ' in opts.enable_targets:
+        if opts.enable_targets == '':
+            enable_target_names = []
+        elif ' ' in opts.enable_targets:
             enable_target_names = opts.enable_targets.split()
         else:
             enable_target_names = opts.enable_targets.split(';')
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3428.1.patch
Type: text/x-patch
Size: 650 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140418/56a9daee/attachment.bin>


More information about the llvm-commits mailing list