[PATCH] D10665: [CMake] Cleanup tools/CMakeLists.txt to take advantage of the auto-registration that was already partially working.

[Jingyi Wei]

wjywbs added a subscriber: wjywbs.
wjywbs added a comment.

I'm not exactly sure but I think this commit broke cmake when extra, llgo, lld and lldb are not checked out. Could you please check the following error log?

  CMake Error at cmake/modules/AddLLVM.cmake:710 (add_subdirectory):
    add_subdirectory given source
    "/path/to/llvm/tools/clang/tools/extra" which is not an
    existing directory.
  Call Stack (most recent call first):
    tools/clang/tools/CMakeLists.txt:24 (add_llvm_external_project)
  
  
  CMake Error at cmake/modules/AddLLVM.cmake:710 (add_subdirectory):
    add_subdirectory given source
    "/path/to/llvm/tools/llgo" which is not an existing
    directory.
  Call Stack (most recent call first):
    tools/CMakeLists.txt:36 (add_llvm_external_project)
  
  
  CMake Error at cmake/modules/AddLLVM.cmake:710 (add_subdirectory):
    add_subdirectory given source "/path/to/llvm/tools/lld"
    which is not an existing directory.
  Call Stack (most recent call first):
    tools/CMakeLists.txt:37 (add_llvm_external_project)
  
  
  CMake Error at cmake/modules/AddLLVM.cmake:710 (add_subdirectory):
    add_subdirectory given source
    "/path/to/llvm/tools/lldb" which is not an existing
    directory.
  Call Stack (most recent call first):
    tools/CMakeLists.txt:38 (add_llvm_external_project)


Repository:
  rL LLVM

http://reviews.llvm.org/D10665