[LLVMdev] proposed patch to default to isl-only polly

[Tobias Grosser]

On 11/19/2013 08:50 PM, Jack Howarth wrote:
> Tobias,
>      Can we add something like the following to polly 3.4?
>
> Index: CMakeLists.txt
> ===================================================================
> --- CMakeLists.txt	(revision 195142)
> +++ CMakeLists.txt	(working copy)
> @@ -81,9 +81,14 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PAT
>
>   FIND_PACKAGE(Isl REQUIRED)
>   FIND_PACKAGE(Gmp REQUIRED)
> -FIND_PACKAGE(Cloog)
>   FIND_PACKAGE(Pluto)
>
> +option(POLLY_USE_CLOOG "Build Polly with Cloog support" OFF)
> +if (POLLY_USE_CLOOG)
> +  # Build Cloog support in Polly (default is ISL-only).
> +  FIND_PACKAGE(Cloog)
> +endif(POLLY_USE_CLOOG)

Hi Jack,

thanks for writing a patch for your feature request. The patch itself 
looks good the way it is implemented. Though I wonder why it is needed. 
Polly only uses CLooG if available, so I assume you have a CLooG version 
installed, but you do not want to use it. What is the reason you do not 
want to use the installed CLooG version. Is the CLooG version outdated 
and Polly does not work with it? Or is there another reason for this?

The reason the isl code generation is not yet default is because we do 
not yet support openmp code generation support for it. Would you be OK 
with installing Polly without OpenMP generation support?

Cheers,
Tobias