[llvm] r268048 - cmake: Set LINK_POLLY_INTO_TOOLS to ON (v2)

[Tobias Grosser via llvm-commits]

On Mon, May 2, 2016, at 05:25 PM, Michael Kruse wrote:
> 2016-04-29 17:07 GMT+02:00 Tobias Grosser via llvm-commits
> <llvm-commits at lists.llvm.org>:
> > +if(WITH_POLLY)
> > +  if(NOT EXISTS ${LLVM_MAIN_SRC_DIR}/tools/polly/CMakeLists.txt)
> 
> I think Polly can also be checked out into other directories, when
> changing the option
> LLVM_EXTERNAL_POLLY_SOURCE_DIR

Hi Michael,

I do not think that LLVM_EXTERNAL_POLLY_SOURCE_DIR has ever been
supported, verified to work or used. In fact, before my commit we
already had the code:

if(WITH_POLLY)
  if(NOT EXISTS ${LLVM_MAIN_SRC_DIR}/tools/polly/CMakeLists.txt)
    set(WITH_POLLY OFF)
  endif()
endif(WITH_POLLY)

which seems to make the use of LLVM_EXTERNAL_POLLY_SOURCE_DIR
impossible. We could probably work on supporting this configuration, but
I don't see yet any use case / request that would justify spending time
on this.

Best,
Tobias