[LLVMbugs] [Bug 19614] New: LINK_POLLY_INTO_TOOLS option doesn't propogate to config.h

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=19614

            Bug ID: 19614
           Summary: LINK_POLLY_INTO_TOOLS option doesn't propogate to
                    config.h
           Product: Build scripts
           Version: trunk
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: normal
          Priority: P
         Component: cmake
          Assignee: unassignedbugs at nondot.org
          Reporter: qneill at codeaurora.org
                CC: llvmbugs at cs.uiuc.edu
    Classification: Unclassified

In the top level llvm CMakeLists.txt we have:

  431   # Configure the three LLVM configuration header files.
  432   configure_file(
  433     ${LLVM_MAIN_INCLUDE_DIR}/llvm/Config/config.h.cmake
  434     ${LLVM_INCLUDE_DIR}/llvm/Config/config.h)

Later on:

  518   option(WITH_POLLY "Build LLVM with Polly" ON)
  519   option(LINK_POLLY_INTO_TOOLS "Static link Polly into tools" OFF)

Over in tools/opt/opt.cpp we see LINK_POLLY_INTO_TOOLS used like:

  358   #ifdef LINK_POLLY_INTO_TOOLS
  359     polly::initializePollyPasses(Registry);
  360   #endif
  361   

It looks like because LINK_POLLY_INTO_TOOLS is after the configure of config.h,
then the #ifdef is never true.

Moving the option up above seemed to fix the problem.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20140430/ac226bd5/attachment.html>