This is an archive of the discontinued LLVM Phabricator instance.

[test] Add libLTO as a clang test dependency on Darwin
ClosedPublic

Authored by vsk on Aug 30 2016, 10:40 AM.

Details

Summary

Running 'check-clang' on a stock checkout of llvm+clang doesn't work on Darwin, because test/Driver/darwin-ld-lto.c can't find libLTO.dylib. Add libLTO as a clang test dependency on Darwin to fix the problem.

Note: We don't have this issue with check-all because libLTO is in the test-depends target.

Diff Detail

Repository
rL LLVM

Event Timeline

vsk updated this revision to Diff 69723.Aug 30 2016, 10:40 AM
vsk retitled this revision from to [test] Add libLTO as a clang test dependency on Darwin.
vsk updated this object.
vsk added a reviewer: bruno.
vsk added a subscriber: cfe-commits.
bruno accepted this revision.Aug 30 2016, 11:36 AM
bruno edited edge metadata.

Nice, LGTM

This revision is now accepted and ready to land.Aug 30 2016, 11:36 AM
This revision was automatically updated to reflect the committed changes.