Done in r280149 (revert this commit) and r280150 (use fake libLTO.dylib). thanks, vedant
> On Aug 30, 2016, at 1:12 PM, Mehdi Amini <mehdi.am...@apple.com> wrote: > > Alternatively, since you only need the presence of a file named libLTO for > the driver to do its magic, test can create an empty file named libLTO.dylib > and use the -ccc-install-dir to make sure the right flag is passed. > >> On Aug 30, 2016, at 10:40 AM, Vedant Kumar <v...@apple.com> wrote: >> >> vsk created this revision. >> vsk added a reviewer: bruno. >> vsk added a subscriber: cfe-commits. >> Herald added a subscriber: mehdi_amini. >> >> 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. >> >> https://reviews.llvm.org/D24042 >> >> Files: >> test/CMakeLists.txt >> >> Index: test/CMakeLists.txt >> =================================================================== >> --- test/CMakeLists.txt >> +++ test/CMakeLists.txt >> @@ -82,6 +82,10 @@ >> endif() >> endif() >> >> +if(APPLE) >> + list(APPEND CLANG_TEST_DEPS LTO) >> +endif() >> + >> add_custom_target(clang-test-depends DEPENDS ${CLANG_TEST_DEPS}) >> set_target_properties(clang-test-depends PROPERTIES FOLDER "Clang tests") >> >> >> >> <D24042.69723.patch> > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
