[PATCH] D95635: [CMake] Require python 3.6 if enabling LLVM test targets

Christopher Tetreault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 11 15:04:00 PST 2021


ctetreau added a comment.

In D95635#2619804 <https://reviews.llvm.org/D95635#2619804>, @serge-sans-paille wrote:

> would it make sense to syndicate the minimal version in a single variable, say `LLVM_MINIMAL_PYTHON_VERSION`, and use it in several place instead of hard-coding the value across multiple files?

Done. That lets us just do the branch in one place as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95635/new/

https://reviews.llvm.org/D95635



More information about the cfe-commits mailing list