[PATCH] D84511: Fix update_cc_test_checks.py --llvm-bin after D78478

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 24 09:12:15 PDT 2020


arichardson marked an inline comment as done.
arichardson added inline comments.


================
Comment at: clang/test/utils/update_cc_test_checks/lit.local.cfg:24
+# works as expected
+config.substitutions.append(
+    ('%update_cc_test_checks_llvm_bin', "%s %s %s" % (
----------------
MaskRay wrote:
> The substitution is here just to make a test. Can it be tested with existing substitutions?
I can't think of an easy way since we need to invoke update_cc_test_checks.py without the "--clang" argument and we need a substitution for the path to the update script.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84511





More information about the llvm-commits mailing list