This is an archive of the discontinued LLVM Phabricator instance.

[cmake] Enable zlib support on windows
ClosedPublic

Authored by labath on Nov 30 2017, 8:44 AM.

Details

Summary

zlib support was hard-wired to off for (non-cygwin) windows targets.
This disables some features, such as reading debug info from compressed
dwarf sections.

This has been this way since zlib support was added in 2013 (r180083),
but there is no obvious reason for that. Zlib is perfectly capable of
being compiled for windows (it even has a cmake file that works out of
the box).

This enables one to turn on zlib support on windows, if one has zlib
avaliable.

Event Timeline

labath created this revision.Nov 30 2017, 8:44 AM
beanz accepted this revision.Nov 30 2017, 10:51 AM

LGTM!

This revision is now accepted and ready to land.Nov 30 2017, 10:51 AM
rnk accepted this revision.Nov 30 2017, 11:35 AM

lgtm

This revision was automatically updated to reflect the committed changes.
grimar added a subscriber: grimar.Dec 1 2017, 11:46 PM
Hahnfeld added a subscriber: Hahnfeld.EditedDec 4 2017, 7:48 AM

I think this commit broke some buildbots:
http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/7617
http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/6962 (this one was already red at that time, so you probably didn't get a notification)

Edit: Also reported on mailing list: http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20171127/506845.html