[libcxx-commits] [libcxx] [libc++] Fixes lit portability issues. (PR #72435)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Nov 15 15:10:19 PST 2023
================
@@ -29,7 +29,13 @@ def appendToSubstitution(substitutions, key, value):
return [(k, v + " " + value) if k == key else (k, v) for (k, v) in substitutions]
-std = getSubstitution("%{cxx_std}", config)
+# External users of the LLVM test suite might not have set the substitutions.
+std = ""
+try:
+ std = getSubstitution("%{cxx_std}", config)
+except:
+ pass
----------------
ldionne wrote:
But still, my point is that folks need to be running the test suite via the normal entry points which includes `params.py`, and I don't think it makes sense to try to support anything else. Otherwise we're bound to break them on a regular basis and that's not good for anyone. @StephanTLavavej Are you folks going through our normal entry point for Lit or do you still maintain your own custom Lit test format?
https://github.com/llvm/llvm-project/pull/72435
More information about the libcxx-commits
mailing list