[PATCH] D45272: [UpdateTestChecks] Add update_analyze_test_checks.py for cost model analysis generation

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 4 12:21:52 PDT 2018


RKSimon added a comment.

In https://reviews.llvm.org/D45272#1057303, @MaskRay wrote:

> Shall we use `#!/usr/bin/env python3` for newer utility scripts?
>
> I know https://llvm.org/docs/GettingStarted.html#software specifies `>=2.7` (presumably Python 2) and some infrastructure rely on Python 2. But these utility scripts are independent and can be migrated to Python 3. https://pythonclock.org/
>
> `utils/UpdateTestChecks/*.py` are Python 2/3 compatible but `utils/update_{,llc_}test_checks.py` uses `python2.7` because of reviewers' stickness (when I refactored the code) with Python 2.


I have no preference either way for this - naturally this would definitely mean that it isn't merged into update_test_checks.py


Repository:
  rL LLVM

https://reviews.llvm.org/D45272





More information about the llvm-commits mailing list