[PATCH] D70660: Add initial tests for update_{llc,cc}_test_checks.py
Alexander Richardson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 16 11:40:05 PST 2019
arichardson added a comment.
In D70660#1786330 <https://reviews.llvm.org/D70660#1786330>, @thakis wrote:
> I added a REQUIRES: line (and did the greppability suggestion) in cc802ea67beb66 <https://reviews.llvm.org/rGcc802ea67beb66d2f8a935e647c3aedcf7848211>. I'd be interested in ways to make these tests not depend on py3 though.
We should be able to use `sys.executable` instead if its python3.
I'm not sure how much longer python 2 will be used by default for running tests. Once that changes we can use sys.executable unconditionally.
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