This is an archive of the discontinued LLVM Phabricator instance.

[test] Extend llvm_shlib_dir fix to unittests
ClosedPublic

Authored by mgorny on Dec 15 2016, 8:11 AM.

Details

Summary

Extend the fix from rL286952 to unittests. The fix added clang built
library directories (via llvm_shlib_dir) to LD_LIBRARY_PATH.
The previous logic has used llvm_libs_dir only which points to installed
LLVM when doing stand-alone builds.

The patch also removes the redundant win32 code that is no longer
necessary now that shlibdir is used unconditionally.

Diff Detail

Repository
rL LLVM

Event Timeline

mgorny updated this revision to Diff 81587.Dec 15 2016, 8:11 AM
mgorny retitled this revision from to [test] Extend llvm_shlib_dir fix to unittests.
mgorny updated this object.
mgorny added a subscriber: cfe-commits.
beanz accepted this revision.Dec 15 2016, 11:22 AM
beanz edited edge metadata.

Looks reasonable to me.

This revision is now accepted and ready to land.Dec 15 2016, 11:22 AM
This revision was automatically updated to reflect the committed changes.