[LLVMdev] [cfe-dev] is configure+make dead yet?

Óscar Fuentes ofv at wanadoo.es
Fri Jun 29 08:21:29 PDT 2012


hume npx <humeafo at gmail.com> writes:

> *hi,Óscar:*
> *
> *
> *so following patch should address both the relocation problem and
> uninstall tree problem, not fully tested just for discussion.*
> *
> *
> Index: LLVMConfig.cmake.in
> ===================================================================
> --- LLVMConfig.cmake.in (revision 159425)
> +++ LLVMConfig.cmake.in (working copy)
> @@ -32,8 +32,11 @@
>  set(LLVM_ON_WIN32 @LLVM_ON_WIN32@)
>
>  set(LLVM_INSTALL_PREFIX @LLVM_INSTALL_PREFIX@)
> -set(LLVM_INCLUDE_DIRS ${LLVM_INSTALL_PREFIX}/include)
> -set(LLVM_LIBRARY_DIRS ${LLVM_INSTALL_PREFIX}/lib)
> +
> +get_filename_component(LLVM_CURRENT_LIST_DIR CMAKE_CURRENT_LIST_FILE PATH)
> +set(LLVM_INCLUDE_DIRS ${LLVM_CURRENT_LIST_DIR}/../../../include)
> +set(LLVM_LIBRARY_DIRS ${LLVM_CURRENT_LIST_DIR}/../../../lib)
> +

What about LLVM_TOOLS_BINARY_DIR ?

Have you checked that CMAKE_CURRENT_LIST_FILE is available for CMake
2.8.0 ?

>  set(LLVM_DEFINITIONS "-D__STDC_LIMIT_MACROS" "-D__STDC_CONSTANT_MACROS")
>
>  # We try to include using the current setting of CMAKE_MODULE_PATH,
> @@ -47,5 +50,7 @@
>      ${CMAKE_MODULE_PATH}
>      "@LLVM_SOURCE_DIR@/cmake/modules")
>    include( LLVM-Config )
> +
> +  set(LLVM_INCLUDE_DIRS ${LLVM_INCLUDE_DIRS} "@LLVM_SOURCE_DIR@/include")
>  endif()

Once you address the two questions above, please upload the patch to the
bugzilla ticket you opened for this bug. Thanks!




More information about the llvm-dev mailing list