This is an archive of the discontinued LLVM Phabricator instance.

Check for Haiku when setting LIB_NAMES for GNU ld
ClosedPublic

Authored by korli on Jul 5 2017, 1:03 AM.

Details

Summary

Haiku uses GNU ld for linking, but is not captured in the conditional when setting LIB_NAMES. This causes a shared library with no symbols on Haiku.
This patch simply adds a check for whether the CMake system name is Haiku in addition to the existing checks.

Diff Detail

Repository
rL LLVM

Event Timeline

korli created this revision.Jul 5 2017, 1:03 AM
axw accepted this revision.Jul 6 2017, 7:38 PM
This revision is now accepted and ready to land.Jul 6 2017, 7:38 PM
korli added a comment.Jul 10 2017, 7:37 AM

I haven't commit access. Could you commit the change?

This revision was automatically updated to reflect the committed changes.