[llvm-commits] [llvm] r161295 - /llvm/trunk/cmake/modules/AddLLVM.cmake

[NAKAMURA Takumi]

Author: chapuni
Date: Fri Aug  3 21:14:44 2012
New Revision: 161295

URL: http://llvm.org/viewvc/llvm-project?rev=161295&view=rev
Log:
[CMake] add_lit_target: Remove comments about add_dependencies. It is not a bug in cmake that add_custom_target(DEPENDS) would not accept targets but file-level dependencies.

Modified:
    llvm/trunk/cmake/modules/AddLLVM.cmake

Modified: llvm/trunk/cmake/modules/AddLLVM.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddLLVM.cmake?rev=161295&r1=161294&r2=161295&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddLLVM.cmake (original)
+++ llvm/trunk/cmake/modules/AddLLVM.cmake Fri Aug  3 21:14:44 2012
@@ -271,11 +271,6 @@
     COMMAND ${LIT_COMMAND} ${ARG_DEFAULT_ARGS}
     COMMENT "${comment}"
     )
-
-  # It would be nice to use the DEPENDS clause in add_custom_target above, but
-  # that has bugs with the CMake 2.8.0 installed on Ubuntu Lucid when the entry
-  # in the depends is another custom target. Instead we add them through an
-  # explicit add_dependencies.
   add_dependencies(${target} ${ARG_DEPENDS})
 endfunction()