Looks like we need something like this:
May 19 2020
Ha shoot, looks like you're fixing this in https://github.com/llvm/llvm-project/commit/f8cccd126b4780a42c00ba84848a66a1c8567183 ?
Thanks! I don't think that I can commit this though, the doc says "If you do not have commit access, someone has to commit the change for you (with attribution)" , could you please do that? Thanks!
Thanks for the patch! The way I thought this would look is to have the arg in http://llvm-cs.pcc.me.uk/utils/gn/build/libs/zlib/enable.gni#3 and then change the lower declare_args block to have llvm_enable_zlib = host_os != "win" || zlib_path != "". Does that make sense?
Address Nico's comments.
May 15 2020