[clang] [Darwin][Driver][clang] Prioritise command line args over `DEFAULT_SYSROOT` (PR #115993)

Carlo Cabrera via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 9 20:58:51 PST 2024


carlocab wrote:

> > Sorry, which env var are you referring to?
> 
> I'm suggesting an explicit test case that checks that `DEFAULT_SYSROOT` is set and behaves as expected. It's ignored if sysroot or isysroot is passed, it's used when they're missing.

`DEFAULT_SYSROOT` is not an environment variable. It is a CMake variable/macro that must be set at configure-time:

https://github.com/llvm/llvm-project/blob/aac000a01ba8c0ed15fec3c198fce9f13fc83725/clang/CMakeLists.txt#L208-L209

https://github.com/llvm/llvm-project/blob/aac000a01ba8c0ed15fec3c198fce9f13fc83725/clang/include/clang/Config/config.h.cmake#L48-L49

I don't think `lit` knows how to tell whether `DEFAULT_SYSROOT` is set. We could maybe teach it to, but given that that intent of #94284 is to deprecate it (and I think all usage of it in existing builders has been removed), that seems like the wrong direction.

https://github.com/llvm/llvm-project/pull/115993


More information about the cfe-commits mailing list