]> granicus.if.org Git - clang/commit
Merging r366447:
authorHans Wennborg <hans@hanshq.net>
Mon, 26 Aug 2019 12:31:59 +0000 (12:31 +0000)
committerHans Wennborg <hans@hanshq.net>
Mon, 26 Aug 2019 12:31:59 +0000 (12:31 +0000)
commit42308c2a02f29bae72439c7bb171c36e974a9064
tree230ec5a46ea42a838a14ba52d1916f4127d74e58
parent2953b4783ad59e72986bdc7c45ab150d4acca237
Merging r366447:
------------------------------------------------------------------------
r366447 | jdevlieghere | 2019-07-18 17:17:42 +0200 (Thu, 18 Jul 2019) | 19 lines

[CMake] Don't set Python_ADDITIONAL_VERSIONS

Until recently, Python_ADDITIONAL_VERSIONS was used to limit LLVM's
Python support to 2.7. Now that both LLVM and LLDB both support Python
3, there's no longer a need to put an arbitrary limit on this.

However, instead of removing the variable, r365692 expanded the list,
which has the (presumably unintentional) side-effect of expression
preference for Python 3.

Instead, as Michal proposed in the original code review, we should just
not set the list at all, and let CMake pick whatever Python interpreter
you have in your path.

This patch removes the Python_ADDITIONAL_VERSIONS variable in llvm,
clang and lld. I've also updated the docs with the default behavior and
how to force a different Python version to be used.

Differential revision: https://reviews.llvm.org/D64894
------------------------------------------------------------------------

git-svn-id: https://llvm.org/svn/llvm-project/cfe/branches/release_90@369900 91177308-0d34-0410-b5e6-96231b3b80d8
CMakeLists.txt