On Mon, Jul 25, 2011 at 6:10 PM, Óscar Fuentes <span dir="ltr"><<a href="mailto:ofv@wanadoo.es">ofv@wanadoo.es</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

> Index: tools/llvmc/src/CMakeLists.txt<br>
> ===================================================================<br>
> --- tools/llvmc/src/CMakeLists.txt    (revision 0)<br>
> +++ tools/llvmc/src/CMakeLists.txt    (revision 0)<br>
> @@ -0,0 +1,25 @@<br>
> +set(LLVMGCCCOMMAND ${CMAKE_BINARY_DIR}/bin/clang)<br>
> +set(LLVMGXXCOMMAND ${CMAKE_BINARY_DIR}/bin/clang++)<br>
<br>
The above will stop working once LLVM is installed and the build<br>
directory removed, or when clang is not included in the build.<br>
<br>
Maybe<br>
<br>
set(LLVMGCCCOMMAND "${CMAKE_C_COMPILER}")<br>
set(LLVMGCCCOMMAND "${CMAKE_CXX_COMPILER}")<br>
<br>
?<br></blockquote><div> </div><div>CMAKE_C_COMPILER is not guaranteed to be able to -emit-llvm. As i've understand, LLVM{GCC,GXX}COMMAND should point to  clang, llvm-gcc or dragonegg.<br>I've built LLVM with autotools and ran llvmc. It seems, LLVM{GCC,GXX}COMMAND aren't contain absolute path, but just defaults to llvm-g{cc,++}. And there is -clang option for llvmc, which changes llvm-g{cc,++} to clang{++}. So, the soultion here is just<br>

<br>set(LLVMGCCCOMMAND llvm-gcc)<br>
set(LLVMGCCCOMMAND llvm-g++)<br><br>But since llvm-gcc is almost dead, isn't it better to make clang default? Of course, this would require changes in Makefiles for consistency and in llvmc itself for option update.<br>

<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> +set(LLVM_USED_LIBS CompilerDriver)<br>
<br>
Why do you neeed CompilerDriver ?<br></blockquote><div><br>Because llvmc depends on it. In llvmc/src/Makefile there is line<br><br>LLVMC_BASED_DRIVER = llvmc<br><br>and in Makefile.rules there are following lines<br><br>

ifdef LLVMC_BASED_DRIVER<br><br>LLVMLIBS = CompilerDriver.a<br><br>At last, removing set(LLVM_USED_LIBS CompilerDriver) causes linking failures.<br><br>I've adressed your notes in updated patch.<br></div></div>