[PATCH] D24042: [test] Add libLTO as a clang test dependency on Darwin

[Vedant Kumar via cfe-commits]

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.amini at 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 <vsk at 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>
>