[all-commits] [llvm/llvm-project] 68a235: build: reduce CMake handling for zlib

Saleem Abdulrasool via All-commits all-commits at lists.llvm.org
Wed Jan 1 16:37:10 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 68a235d07f9e7049c7eb0c8091f37e385327ac28
      https://github.com/llvm/llvm-project/commit/68a235d07f9e7049c7eb0c8091f37e385327ac28
  Author: Saleem Abdulrasool <compnerd at compnerd.org>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M clang/test/CMakeLists.txt
    M clang/test/lit.site.cfg.py.in
    M compiler-rt/test/lit.common.configured.in
    M lld/test/CMakeLists.txt
    M lld/test/lit.site.cfg.py.in
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
    M llvm/cmake/config-ix.cmake
    M llvm/include/llvm/Config/config.h.cmake
    M llvm/lib/Support/CMakeLists.txt
    M llvm/lib/Support/CRC.cpp
    M llvm/lib/Support/Compression.cpp
    M llvm/test/CMakeLists.txt
    M llvm/test/lit.site.cfg.py.in
    M llvm/unittests/Support/CompressionTest.cpp

  Log Message:
  -----------
  build: reduce CMake handling for zlib

Rather than handling zlib handling manually, use `find_package` from CMake
to find zlib properly. Use this to normalize the `LLVM_ENABLE_ZLIB`,
`HAVE_ZLIB`, `HAVE_ZLIB_H`. Furthermore, require zlib if `LLVM_ENABLE_ZLIB` is
set to `YES`, which requires the distributor to explicitly select whether
zlib is enabled or not. This simplifies the CMake handling and usage in
the rest of the tooling.




More information about the All-commits mailing list