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: [email protected]
          Reporter: [email protected]
                CC: [email protected]
    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.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to