[PATCH] D27386: build: add support for standalone lld build

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 07:19:08 PST 2016


mgorny added inline comments.


================
Comment at: CMakeLists.txt:6
+
+  if(NOT LLVM_CONFIG_PATH)
+    find_program(LLVM_CONFIG_PATH "llvm-config" DOC "Path to llvm-config binary")
----------------
Is there a reason for an additional conditional like this? `find_program` makes LLVM_CONFIG_PATH a cache variable, so I think this would be cleaner without the outer conditional.


Repository:
  rL LLVM

https://reviews.llvm.org/D27386





More information about the llvm-commits mailing list