[PATCH] D77184: Make it possible for lit.site.cfg to contain relative paths, and use it for llvm and clang

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 2 10:50:11 PDT 2020


thakis marked 4 inline comments as done.
thakis added a comment.

Thanks!



================
Comment at: llvm/cmake/modules/AddLLVM.cmake:1483
+
+  set(arg_path_limit 0)
+  if (ARG_PATH_VALUES)
----------------
hans wrote:
> Maybe comment what this does?
Turns out the saving and undoing isn't needed since cmake copies all variables to new scopes, and this changes just the local copy. So this is gone now.


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

https://reviews.llvm.org/D77184





More information about the llvm-commits mailing list