[PATCH] D79151: build: use `find_package(Python3)` if available (llvm runtimes).

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 13 16:57:09 PDT 2020


compnerd added inline comments.


================
Comment at: llvm/runtimes/CMakeLists.txt:118
+    include(FindPythonInterp)
+    if( NOT PYTHONINTERP_FOUND )
+      message(FATAL_ERROR
----------------
This was not fatal previously was it?  Why make it fatal now?


================
Comment at: llvm/runtimes/CMakeLists.txt:139
+      if(Python2_VERSION VERSION_LESS 2.7)
+        message(SEND_ERROR "Python 2.7 or newer is required")
+      endif()
----------------
Similar


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79151





More information about the llvm-commits mailing list