[PATCH] D33299: [Polly][CMake] Use the CMake Package instead of llvm-config in out-of-tree builds

[Philip Pfaffe via Phabricator via llvm-commits]

philip.pfaffe added a comment.

In https://reviews.llvm.org/D33299#761863, @Meinersbur wrote:

> My first tryout failed with this error:


When importing a package you automatically get <PACK>_DIR variable pointing to the directory where the package Config is located. I.e., setting LLVM_DIR to ${LLVM_BUILD_TREE}/lib/cmake/llvm is what you want to do.

> Next try by setting `CMAKE_PREFIX_PATH`:

This works, too, however setting the LLVM_DIR variable instead usually is cleaner.

> Polly is in my source checkout, so it got included automatically. I can switch it off using `-DLLVM_TOOL_POLLY_BUILD=OFF`. Of course it is not intended to build Polly externally if Polly is already available in LLVM, but it means I have to build some more LLVM builds (static/shared/dylib) around to test this.

I understand. I'd expect switching LLVM_TOOL_POLLY_BUILD to OFF should work.

> With `-DLLVM_TOOL_POLLY_BUILD=OFF` it compiles, but almost all regression tests fail. For instance:

I can't reproduce this.  I've seen similar errors before, though, when I broke ABI compatibilty between builds, e.g. by mixing Debug with Release, or mismatched SVN revisions.


https://reviews.llvm.org/D33299