[PATCH] D66834: Driver tests: set `--sysroot=""` to support clang with `DEFAULT_SYSROOT`

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 30 09:02:06 PDT 2019


compnerd added a comment.

I think that this is pretty easy to forget.  Fortunately, last argument wins.  Why not sink this into the `%clang` substitution in lit?  That ensures that we run with an empty sysroot and then when the test needs to adjust the sysroot, it can do so explicitly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66834





More information about the cfe-commits mailing list