Random question. Is there any reason to generate clang-cl if you’re not on 
Windows?

It may make sense to move clang-cl under the if(WIN32).

-Chris

> On Sep 21, 2015, at 3:44 PM, Hans Wennborg via cfe-commits 
> <cfe-commits@lists.llvm.org> wrote:
> 
> Author: hans
> Date: Mon Sep 21 17:44:57 2015
> New Revision: 248221
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=248221&view=rev
> Log:
> Add msbuild-bin/cl to CLANG_LINKS_TO_CREATE
> 
> This got lost in r248043 which refactored the symlink creation.
> 
> Modified:
>    cfe/trunk/tools/driver/CMakeLists.txt
> 
> Modified: cfe/trunk/tools/driver/CMakeLists.txt
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/driver/CMakeLists.txt?rev=248221&r1=248220&r2=248221&view=diff
> ==============================================================================
> --- cfe/trunk/tools/driver/CMakeLists.txt (original)
> +++ cfe/trunk/tools/driver/CMakeLists.txt Mon Sep 21 17:44:57 2015
> @@ -62,6 +62,10 @@ add_custom_target(install-clang
> 
> if(NOT CLANG_LINKS_TO_CREATE)
>   set(CLANG_LINKS_TO_CREATE clang++ clang-cl)
> +
> +  if (WIN32)
> +    list(APPEND CLANG_LINKS_TO_CREATE ../msbuild-bin/cl)
> +  endif()
> endif()
> 
> foreach(link ${CLANG_LINKS_TO_CREATE})
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to