[PATCH] D21892: Clang modules builds depend header-wise on it as they ship all headers from the umbrella folders.

Vassil Vassilev via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 10:51:18 PDT 2016


v.g.vassilev updated this revision to Diff 62382.
v.g.vassilev added a comment.

Force the dependency only if modules are enabled.


http://reviews.llvm.org/D21892

Files:
  CMakeLists.txt

Index: CMakeLists.txt
===================================================================
--- CMakeLists.txt
+++ CMakeLists.txt
@@ -698,6 +698,15 @@
 
 add_subdirectory(utils/TableGen)
 
+# Force target to be built as soon as possible. Clang modules builds depend
+# header-wise on it as they ship all headers from the umbrella folders. Building
+# an entire module might include header, which depends on intrinsics_gen. This
+# should be right after LLVMSupport and LLVMTableGen otherwise we introduce a
+# circular dependence.
+if (LLVM_ENABLE_MODULES)
+  list(APPEND LLVM_COMMON_DEPENDS intrinsics_gen)
+endif(LLVM_ENABLE_MODULES)
+
 add_subdirectory(include/llvm)
 
 add_subdirectory(lib)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21892.62382.patch
Type: text/x-patch
Size: 693 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160630/cf8c375e/attachment.bin>


More information about the llvm-commits mailing list