[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