This is an archive of the discontinued LLVM Phabricator instance.

[lld] [test] Fix zlib cond when building stand-alone, clean up
ClosedPublic

Authored by mgorny on Jan 31 2017, 11:11 AM.

Details

Summary

Fix the test zlib conditional to use LLVM_ENABLE_ZLIB value when
building stand-alone. The HAVE_LIBZ is not available when performing
a stand-alone build. Since the zlib support is a feature of
the underlying LLVM library, it exports the actual status as the final
value of LLVM_ENABLE_ZLIB in LLVMConfig.

While at it, canonicalize the boolean value into 0/1 and remove unused
CMake definitions (most likely copied from clang).

Diff Detail

Event Timeline

mgorny created this revision.Jan 31 2017, 11:11 AM
mgorny added a reviewer: beanz.Feb 3 2017, 9:14 AM

A gentle ping.

rnk accepted this revision.Feb 8 2017, 10:52 AM

lgtm

This revision is now accepted and ready to land.Feb 8 2017, 10:52 AM
This revision was automatically updated to reflect the committed changes.