[polly] dependence on Intrinsics.gen

[Tobias Grosser]

On 03/10/2014 10:11 PM, Sebastian Pop wrote:
> Hi Tobi,
>
> I am currently seeing this kind of errors when compiling polly with cmake:
>
> In file included from ../include/llvm/IR/IntrinsicInst.h:30:0,
>                   from ../tools/polly/lib/CodeGen/BlockGenerators.cpp:28:
> ../include/llvm/IR/Intrinsics.h:41:34: fatal error: llvm/IR/Intrinsics.gen: No such file or directory
>   #include "llvm/IR/Intrinsics.gen"
>                                    ^
>
>
> In file included from ../include/llvm/IR/IntrinsicInst.h:30:0,
>                   from ../tools/polly/lib/IndVarSimplify.cpp:43:
> ../include/llvm/IR/Intrinsics.h:41:34: fatal error: llvm/IR/Intrinsics.gen: No such file or directory
>   #include "llvm/IR/Intrinsics.gen"
>                                    ^
>
> I think LLVMPollyLib should depend on intrinsics_gen:
>
> diff --git a/lib/CMakeLists.txt b/lib/CMakeLists.txt
> index dc66491..659c4b0 100644
> --- a/lib/CMakeLists.txt
> +++ b/lib/CMakeLists.txt
> @@ -80,7 +80,7 @@ add_polly_loadable_module(LLVMPolly
>
>   if (TARGET intrinsics_gen)
>     # Check if we are building as part of an LLVM build
> -  add_dependencies(LLVMPolly intrinsics_gen)
> +  add_dependencies(LLVMPollyLib intrinsics_gen)
>   endif()
>
>   target_link_libraries(LLVMPolly LLVMPollyLib)
>
>
> as there is no dependence between LLVMPolly and LLVMPollyLib, and LLVMPollyLib
> does contain a dependence on building the .cpp files above.
>
> Ok to commit this change?

Yes, that's an obvious fix.

Tobias