This is an archive of the discontinued LLVM Phabricator instance.

Differential D79295

Make Polly tests dependencies explicit
ClosedPublic

Authored by serge-sans-paille on May 2 2020, 1:22 PM.

Details

Reviewers
Meinersbur
aheejin
bollu
Commits
rG6f71678ecd29: Make Polly tests dependencies explicit
rG8ceee08de135: Make Polly tests dependencies explicit
Summary

Due to libPolly now using the component infrastructure, it no longer carries all dependencies as it used to do.

Diff Detail

Event Timeline

serge-sans-paille created this revision.May 2 2020, 1:22 PM
Herald added a reviewer: bollu. · View Herald TranscriptMay 2 2020, 1:22 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: llvm-commits, mgorny. · View Herald Transcript
Harbormaster failed remote builds in B55557: Diff 261661!May 2 2020, 2:50 PM
Meinersbur accepted this revision.May 2 2020, 7:42 PM

LGTM.

Thanks.

This revision is now accepted and ready to land.May 2 2020, 7:42 PM
aheejin accepted this revision.May 3 2020, 1:10 AM

Thanks! This fixes the shared library problem.

Closed by commit rG8ceee08de135: Make Polly tests dependencies explicit (authored by serge-sans-paille). · Explain WhyMay 3 2020, 11:19 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
polly/
unittests/
ScopPassManager/
2 lines

Diff 261739

polly/unittests/ScopPassManager/CMakeLists.txt

llvm_map_components_to_libnames(llvm_libs Passes Core Analysis)
add_polly_unittest(ScopPassManagerTests add_polly_unittest(ScopPassManagerTests
PassManagerTest.cpp PassManagerTest.cpp
) )
target_link_libraries(ScopPassManagerTests PRIVATE ${llvm_libs})