[PATCH] D13622: Add call to find_package to load LLVM dependencies

don hinton via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 10 11:20:11 PDT 2015


hintonda created this revision.
hintonda added a reviewer: chapuni.
hintonda added a subscriber: cfe-commits.

ClangConfig requires LLVMConfig, so add find_package call in ClangConfig so find_package(clang REQUIRED CONFIG) will just work.  This makes it easier for cmake based projects to use clang, e.g., tools using ClangTooling.

http://reviews.llvm.org/D13622

Files:
  cmake/modules/ClangConfig.cmake

Index: cmake/modules/ClangConfig.cmake
===================================================================
--- cmake/modules/ClangConfig.cmake
+++ cmake/modules/ClangConfig.cmake
@@ -4,5 +4,7 @@
 # uses LLVM's. When it does, we should move this file to ClangConfig.cmake.in
 # and call configure_file() on it.
 
+find_package(LLVM REQUIRED CONFIG)
+
 # Provide all our library targets to users.
 include("${CMAKE_CURRENT_LIST_DIR}/ClangTargets.cmake")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13622.37031.patch
Type: text/x-patch
Size: 452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151010/4dd1c3ed/attachment.bin>


More information about the cfe-commits mailing list