[PATCH] D45369: [UpdateTestChecks] Add update_mca_test_checks.py script

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 12 11:23:39 PDT 2018


RKSimon added a comment.

In https://reviews.llvm.org/D45369#1061961, @MaskRay wrote:

> It would be great to use a python3 shebang. These utilities are independent from other (possibly Python 2 specific) Python files
>
>   % head -n 1 utils/update_cc_test_checks.py
>   #!/usr/bin/env python3
>


We still have problems with some dev machines defaulting to python27 - as long as this file runs fine with both python27/pyjthon3 I don't see keeping to python27 as a problem - if its just flipping the shebangs thats a trivial future change,


https://reviews.llvm.org/D45369





More information about the llvm-commits mailing list