This is an archive of the discontinued LLVM Phabricator instance.

Treat polly as "in tree" if LLVM_EXTERNAL_POLLY_SOURCE_DIR is provided
ClosedPublic

Authored by etherzhhb on Oct 6 2016, 6:34 PM.

Details

Summary

Treat polly as "in tree" if LLVM_EXTERNAL_POLLY_SOURCE_DIR is provided, since in this case polly is also built as if it is in tree.

Diff Detail

Repository
rL LLVM

Event Timeline

etherzhhb updated this revision to Diff 73875.Oct 6 2016, 6:34 PM
etherzhhb retitled this revision from to Treat polly as "in tree" if LLVM_EXTERNAL_POLLY_SOURCE_DIR is provided.
etherzhhb updated this object.
etherzhhb set the repository for this revision to rL LLVM.
etherzhhb added a subscriber: Restricted Project.
Meinersbur edited edge metadata.Oct 7 2016, 2:57 AM

Did you successfully build Polly out-of-tree with this. with an without LLVM_POLLY_LINK_INTO_TOOLS? I think I remember there were other places where it assumed that polly is in ${LLVM_ROOT}/tools/polly.

The only "tools/polly" I saw is :
./CMakeLists.txt:if (EXISTS ${LLVM_MAIN_SRC_DIR}/tools/polly/CMakeLists.txt)

I am setting LLVM_POLLY_LINK_INTO_TOOLS and running check-all

I tried the combination of {LLVM_POLLY_LINK_INTO_TOOLS On/Of} x {cmake from scratch/cmake on existing cmakecach}, and it passed check-all (which includes check-polly).

Meinersbur accepted this revision.Oct 7 2016, 12:36 PM
Meinersbur edited edge metadata.

Well, the other occurrences might have already been removed. Since this works: LGTM

This revision is now accepted and ready to land.Oct 7 2016, 12:36 PM
This revision was automatically updated to reflect the committed changes.