[PATCH] D132870: tweak zstd behavior in cmake and llvm config for better testing

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: llvm/cmake/modules/LLVMConfig.cmake.in:78
 if(LLVM_ENABLE_ZSTD)
-  set(zstd_ROOT @zstd_ROOT@)
+  set(zstd_DIR @zstd_DIR@)
   find_package(zstd)
----------------
I think it should be possible to omit this line altogether, this shouldn't be needed (zlib above is a special case).


================
Comment at: llvm/lib/Support/CMakeLists.txt:24
 
+set(imported_libs "")
+
----------------
This is unnecessary, you can just omit this line.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132870/new/

https://reviews.llvm.org/D132870