[PATCH] D57366: Add zlib feature to compiler-rt

[Matthew Voss via Phabricator via llvm-commits]

ormris created this revision.
ormris added reviewers: eugenis, kcc, beanz.
Herald added a subscriber: dberris.

Add zlib feature to compiler-rt lit config

A new test was added to libfuzzer in r352265 that requires zlib. Our internal
buildbot doesn’t have zlib installed and other projects (llvm, clang) allow us
to add this as a “REQUIRES” flag. This change adds this flag to the compiler-rt
project.


https://reviews.llvm.org/D57366

Files:
  compiler-rt/test/fuzzer/lit.cfg
  compiler-rt/test/fuzzer/lit.site.cfg.in


Index: compiler-rt/test/fuzzer/lit.site.cfg.in
===================================================================
--- compiler-rt/test/fuzzer/lit.site.cfg.in
+++ compiler-rt/test/fuzzer/lit.site.cfg.in
@@ -12,6 +12,8 @@
 config.llvm_library_dir = "@LLVM_LIBRARY_DIR@"
 config.target_triple = "@TARGET_TRIPLE@"
 
+config.have_zlib = "@HAVE_LIBZ@"
+
 # Load common config for all compiler-rt lit tests.
 lit_config.load_config(config,
     "@COMPILER_RT_BINARY_DIR@/test/lit.common.configured")
Index: compiler-rt/test/fuzzer/lit.cfg
===================================================================
--- compiler-rt/test/fuzzer/lit.cfg
+++ compiler-rt/test/fuzzer/lit.cfg
@@ -24,6 +24,9 @@
 # the test runner updated.
 config.test_format = lit.formats.ShTest(execute_external)
 
+if config.have_zlib == "1":
+    config.available_features.add("zlib")
+
 # LeakSanitizer is not supported on OSX or Windows right now.
 if (sys.platform.startswith('darwin') or
     sys.platform.startswith('freebsd') or


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57366.183998.patch
Type: text/x-patch
Size: 1002 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190129/f0addc55/attachment.bin>