phosek added a comment. In D79219#2152747 <https://reviews.llvm.org/D79219#2152747>, @labath wrote:
> I wouldn't mind separate (internal) cache variable, though I am somewhat > surprised by this problem. A (non-cache) variable set in one of the parent > scopes should still take precedence over a cache variable with the same name. > And since config-ix.cmake is included from the top CMakeLists.txt, the value > it defines should be available everywhere. Was this a problem for the regular > build, or only for some of the more exotic build setups that don't start with > llvm/CMakeLists.txt ? Never mind, it's working as expected. The problem is that we disable zlib detection on Windows which I missed before. I'm not sure why that's the case, I tested this on Windows and it seems to be working fine, but for now I've kept the existing behavior, we can consider enabling zlib on Windows in a follow up change. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79219/new/ https://reviews.llvm.org/D79219 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
