ckissane added a comment.
In D128465#3642997 <https://reviews.llvm.org/D128465#3642997>, @MaskRay wrote:
> As I mentioned, the proper approach is to add zstd functionality along with
> the CMake change, instead of adding CMake to all llvm-project components
> without a way to test them.
@MaskRay, I have now done this and ran the ldd tests as requested:
With LLVM_ENABLE_ZSTD=ON
$ ninja check-lld
Testing Time: 8.98s
Unsupported : 17
Passed : 2638
Expectedly Failed: 1
With LLVM_ENABLE_ZSTD=OFF
$ ninja check-lld
Testing Time: 8.95s
Unsupported : 17
Passed : 2638
Expectedly Failed: 1
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128465/new/
https://reviews.llvm.org/D128465
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
