[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