This is an archive of the discontinued LLVM Phabricator instance.

Have llvm-c-test only use libLLVM if libLLVM has all the right components.
ClosedPublic

Authored by beanz on Dec 15 2014, 2:38 PM.

Details

Summary

We should only have llvm-c-test use libLLVM if the library is built with the default set of components or if LLVM_DYLIB_COMPONENTS includes all the LLVM_LINK_COMPONENTS required for llvm-c-test. Making libLLVM always used causes build failures if libLLVM doesn't include all

Diff Detail

Repository
rL LLVM

Event Timeline

beanz updated this revision to Diff 17306.Dec 15 2014, 2:38 PM
beanz retitled this revision from to Have llvm-c-test only use libLLVM if libLLVM has all the right components..
beanz updated this object.
beanz edited the test plan for this revision. (Show Details)
beanz added a reviewer: chapuni.
beanz added a subscriber: Unknown Object (MLST).
ributzka accepted this revision.Dec 18 2014, 9:16 AM
ributzka added a reviewer: ributzka.
ributzka added a subscriber: ributzka.

LGTM

This revision is now accepted and ready to land.Dec 18 2014, 9:16 AM
This revision was automatically updated to reflect the committed changes.