This is an archive of the discontinued LLVM Phabricator instance.

[llvm] CYGWIN: Fix build of some plugins/library/unittests
ClosedPublic

Authored by carlo-bramini on Jul 9 2023, 11:01 AM.

Details

Summary

CYGWIN uses the same format of WIN32 for shared libraries.
As the comment says: "a shared library can't have undefined references", in this case CYGWIN must be handled like WIN32 by CMakeLists.txt, otherwise you will get several errors when linking because some symbols are undefined.
Attached patch fixes this issue and allows to complete the build process for those targets.

Diff Detail

Event Timeline

carlo-bramini created this revision.Jul 9 2023, 11:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2023, 11:01 AM
Herald added a subscriber: mtrofin. · View Herald Transcript
carlo-bramini requested review of this revision.Jul 9 2023, 11:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2023, 11:01 AM
mstorsjo accepted this revision.Jul 9 2023, 12:48 PM

LGTM

What's the preferred git author for the commit, Real Name <email@address>?

This revision is now accepted and ready to land.Jul 9 2023, 12:48 PM

LGTM

What's the preferred git author for the commit, Real Name <email@address>?

Real name: Carlo Bramini
email: carlo_bramini@users.sourceforge.net

This revision was automatically updated to reflect the committed changes.