Move version control macros, find_first_existing_file and
find_first_existing_vc_file to AddLLVM.cmake so they can be reused by sub projects
like clang.
Please see D35533 for an example of where this would be useful.
Differential D36971
[CMAKE] Move version control macros to AddLLVM.cmake so they can be reused by clang, etc. hintonda on Aug 21 2017, 10:00 AM. Authored by
Details Move version control macros, find_first_existing_file and Please see D35533 for an example of where this would be useful.
Diff Detail
Event TimelineComment Actions LGTM I just checked that this patch is working correctly from clang. I am just wondering if there is a possibility of using clang only without llvm. If this is the case, clang might need its own version control macros ? Or no such case. Comment Actions
LLVM is required to build clang, either in-tree or found via llvm-config for out-of-tree builds. For in-tree builds, AddLLVM.cmake gets included by llvm/CMakeLists.txt, whereas it will be included by clang/CMakeLists.txt for out of tree builds. Comment Actions I have tested this patch with git, git-svn, svn and repo. It is working as expected with clang's lib/Basic/CMakeLists.txt. |