[PATCH] D19711: [Polly] cmake: Set LINK_POLLY_INTO_TOOLS to ON

[Johannes Doerfert via llvm-commits]

I link Polly into my tools anyway, thus I'm in favor of changing the
default.

> grosser created this revision.
> grosser added reviewers: jdoerfert, Meinersbur, sebpop, etherzhhb, zinob, hiraditya.
> grosser added subscribers: llvm-commits, pollydev.
> 
> With Polly checked into LLVM it was since a long time possible to compile
> clang/opt/bugpoint with Polly support directly linked in, instead of only
> providing Polly as a separate loadable module. This commit switches the
> default from providing Polly as a module to linking Polly into tools, such
> that it becomes unnecessary to load the Polly module when playing with Polly.
> Such configuration has shown a lot more convenient for day-to-day Polly use.
> 
> This change does not impact the default behavior of any tool, if Polly is not
> explicitly enabled when calling clang/opt/bugpoint Polly does not affect
> compilation.
> 
> This change also does not impact normal LLVM/clang checkouts that do not
> contain Polly.
> 
> http://reviews.llvm.org/D19711
> 
> Files:
>   CMakeLists.txt
> 
> Index: CMakeLists.txt
> ===================================================================
> --- CMakeLists.txt
> +++ CMakeLists.txt
> @@ -327,7 +327,7 @@
>    "Use -gsplit-dwarf when compiling llvm." OFF)
>  
>  option(WITH_POLLY "Build LLVM with Polly" ON)
> -option(LINK_POLLY_INTO_TOOLS "Static link Polly into tools" OFF)
> +option(LINK_POLLY_INTO_TOOLS "Static link Polly into tools" ON)
>  
>  # Define an option controlling whether we should build for 32-bit on 64-bit
>  # platforms, where supported.
> 
> 

> Index: CMakeLists.txt
> ===================================================================
> --- CMakeLists.txt
> +++ CMakeLists.txt
> @@ -327,7 +327,7 @@
>    "Use -gsplit-dwarf when compiling llvm." OFF)
>  
>  option(WITH_POLLY "Build LLVM with Polly" ON)
> -option(LINK_POLLY_INTO_TOOLS "Static link Polly into tools" OFF)
> +option(LINK_POLLY_INTO_TOOLS "Static link Polly into tools" ON)
>  
>  # Define an option controlling whether we should build for 32-bit on 64-bit
>  # platforms, where supported.


-- 

Johannes Doerfert
Researcher / PhD Student

Compiler Design Lab (Prof. Hack)
Saarland University, Computer Science
Building E1.3, Room 4.31

Tel. +49 (0)681 302-57521 : doerfert at cs.uni-saarland.de
Fax. +49 (0)681 302-3065  : http://www.cdl.uni-saarland.de/people/doerfert
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 213 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160429/dfcfa23a/attachment.sig>