[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 443035. ckissane added a comment. start breaking this up further, this is now the add cmake config patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files:

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

MaskRay added inline comments. Comment at: llvm/cmake/modules/FindZSTD.cmake:1 +# Copyright (c) Meta Platforms, Inc. and affiliates. +# How did you derive this? The file seems contributed by you (I don't think facebook/zstd has such a file). It should not have

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 441114. ckissane edited the summary of this revision. ckissane added a comment. - feat: add zstd compression - Merge branch 'ckissane.refactor-compression-namespace' into ckissane.add-zstd-compression - Merge branch 'ckissane.refactor-compression-namespace'

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 441102. ckissane added a comment. add AlgorithmName to compression namespaces CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440747. ckissane edited the summary of this revision. ckissane added a comment. make it the second stage CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

MaskRay added a comment. In D128465#3612948 , @phosek wrote: > I think this patch should be broken into at least two: > > 1. Refactor `llvm/include/llvm/Support/Compression.h` and > `llvm/lib/Support/Compression.cpp` to introduce a generic interface and

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

phosek added a comment. I think this patch should be broken into at least two: 1. Refactor `llvm/include/llvm/Support/Compression.h` and `llvm/lib/Support/Compression.cpp` to introduce a generic interface and use it throughout the codebase. 2. zstd support in

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440325. ckissane added a comment. more context CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt clang-tools-extra/clangd/index/Serialization.cpp

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440327. ckissane added a comment. format CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt clang-tools-extra/clangd/index/Serialization.cpp

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440298. ckissane added a comment. fix type for FindZstd cmake usage Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/ https://reviews.llvm.org/D128465 Files: clang-tools-extra/clangd/CMakeLists.txt

[Lldb-commits] [PATCH] D128465: Zstandard as a second compression method to LLVM

ckissane updated this revision to Diff 440288. ckissane added a comment. Fix missing cmake findZstd, remove unused crc32 from compression namespaces and simplify some code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128465/new/