[PATCH] D129324: [llvm] add compression AlgorithmName (s) to the zlib and zstd namespaces

Cole Kissane via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 17 12:32:19 PDT 2022


ckissane added a subscriber: MaskRay.
ckissane added a comment.

@dblaikie Yes, the end goal is to make it configurable and eventually refactor to make a shared "compression class" interface where we have zlib and zstd implementations.
However, this patch is only pointed at adding an implementation of code to call zstd in the first place. Also @MaskRay has an draft patch that demonstrates how this current setup could still be used in a runtime way.
Note to make that intention clear we have dropped the usage of namespace aliases at the request of @phosek and others.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129324



More information about the llvm-commits mailing list