[PATCH] D70660: Add initial tests for update_{llc,cc}_test_checks.py

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 16 08:06:41 PST 2019


thakis added a comment.

> Looks like the test fails on macOS: http://45.33.8.238/mac/4235/step_11.txt

^ Sorry, please ignore that part. I'm still looking if the failure isn't my fault with the bot setup. I meant to delete this after I wrote the py3 question.



================
Comment at: llvm/test/tools/UpdateTestChecks/lit.local.cfg:49
+    config.substitutions.append(('%python3', shell_quote(py3_exe)))
+    config.available_features.add('python3')
+
----------------
Nothing in this patch seems to use this as far as I can tell? Do you need this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70660





More information about the llvm-commits mailing list