[flang-commits] [PATCH] D80377: [flang] Google test infrastructure support for unittests

[Eric Schweitz via Phabricator via flang-commits]

schweitz added inline comments.


================
Comment at: flang/test/Unit/lit.cfg.py:33
+# Propagate PYTHON_EXECUTABLE into the environment
+config.environment['PYTHON_EXECUTABLE'] = sys.executable
+
----------------
richard.barton.arm wrote:
> Other sub-projects set this up in top-level CMakeLists using FindPythonInterp. Why do we set it here?
Sameeran was pulling this glue code from the lld subproject, where these config settings are made. They can probably be deleted here. I do not appear to be used, but I haven't tested it.


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

https://reviews.llvm.org/D80377