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

Greg Bedwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 04:57:48 PDT 2018


gbedwell added a comment.

Yeah.  LGTM too.  In general we should probably put some effort into refactoring the scripts so that they work with both Python versions but it seems when the default was discussed a couple of months back ( http://lists.llvm.org/pipermail/llvm-dev/2018-February/thread.html#120907 ) there were still too many OSs in use shipping with only 2.7 by default, so it makes sense to leave this in line with the existing scripts until there can be a coordinated effort to switch.


Repository:
  rL LLVM

https://reviews.llvm.org/D45272





More information about the llvm-commits mailing list