[libcxx-commits] [PATCH] D97452: [libcxx] [test] Pass some windows environment variables through to test processes

Martin Storsjö via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Feb 25 04:31:03 PST 2021


mstorsjo added a comment.

In D97452#2587321 <https://reviews.llvm.org/D97452#2587321>, @curdeius wrote:

> LGTM. But I believe that the long term solution is to run all the tests in the internal lit shell.

Yes, but I believe that aspect is orthogonal to this script. Currently the lit test driver runs the test script (which consists of two lines, one "clang++" to compile the test and one which invokes the run.py script with python to execute the built test executable) with bash, which maybe could/should be changed to use the lit internal shell. That still keeps the run.py wrapper script for all the purposes it serves right now (codesign, setting up other test specific env variables etc).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97452



More information about the libcxx-commits mailing list