[PATCH] D135437: [Tests] Require zlib for zstd tests

[James Henderson via Phabricator via llvm-commits]

jhenderson added reviewers: MaskRay, ckissane, leonardchan, phosek.
jhenderson added a comment.
Herald added a subscriber: StephenFan.

I'm fairly confident this is the wrong fix for the problem. ZSTD compression shouldn't require ZLIB support (and similarly ZLIB compression shouldn't require ZSTD support). Without looking too hard at the problem, my guess is that there's a bug in the check for whether the --compress-debug-sections option is supported or not. There may also be a lit bug, in that the REQUIRES directive for these compression formats should be based on what happened at cmake configuration time, not (only) on what's installed on the system, but I'm not really sure.

I haven't been involved with the recent compression changes really, so I've added a few people who have.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135437