[all-commits] [llvm/llvm-project] 1fc501: [libc++] Consider everything inside %T to be a dep...

Louis Dionne via All-commits all-commits at lists.llvm.org
Wed Jun 10 19:39:48 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 1fc5010d6b70bb5c2330595230ca5c5fe07bcad0
      https://github.com/llvm/llvm-project/commit/1fc5010d6b70bb5c2330595230ca5c5fe07bcad0
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-06-10 (Wed, 10 Jun 2020)

  Changed paths:
    M libcxx/test/libcxx/double_include.sh.cpp
    M libcxx/test/libcxx/language.support/support.dynamic/libcpp_deallocate.sh.cpp
    M libcxx/test/libcxx/language.support/support.rtti/type.info/type_info.comparison.merged.sh.cpp
    M libcxx/test/libcxx/language.support/support.rtti/type.info/type_info.comparison.unmerged.sh.cpp
    M libcxx/test/libcxx/selftest/dsl/lit.local.cfg
    M libcxx/test/libcxx/selftest/exec.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/convenience_substitutions/build_run.sh.cpp
    A libcxx/test/libcxx/selftest/newformat/file_dependencies/a.txt
    A libcxx/test/libcxx/selftest/newformat/file_dependencies/absolute-and-relative-paths.sh.cpp
    A libcxx/test/libcxx/selftest/newformat/file_dependencies/dir/b.txt
    R libcxx/test/libcxx/selftest/newformat/file_dependencies/no-dependencies.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/file_dependencies/substitute-in-dependencies.sh.cpp
    R libcxx/test/libcxx/selftest/newformat/file_dependencies/substitute-in-run.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/remote-substitutions.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/sh.cpp/substitutions.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/sh.cpp/werror.sh.cpp
    M libcxx/test/libcxx/selftest/newformat/shell-escape-pipes.sh.cpp
    R libcxx/test/libcxx/selftest/test.file_dependencies.sh.cpp
    M libcxx/test/libcxx/selftest/test.sh.cpp
    M libcxx/test/libcxx/strings/basic.string/PR42676.sh.cpp
    M libcxx/test/libcxx/utilities/function.objects/func.blocks.sh.cpp
    M libcxx/test/std/containers/associative/map/PR28469_undefined_behavior_segfault.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/cerr.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/cin.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/clog.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/cout.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcerr.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcin.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wclog.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcout.sh.cpp
    M libcxx/test/std/thread/thread.condition/thread.condition.condvarany/wait_terminates.sh.cpp
    M libcxx/utils/libcxx/test/config.py
    M libcxx/utils/libcxx/test/dsl.py
    M libcxx/utils/libcxx/test/newformat.py
    M libcxx/utils/run.py
    M libcxx/utils/ssh.py
    M libcxxabi/test/incomplete_type.sh.cpp

  Log Message:
  -----------
  [libc++] Consider everything inside %T to be a dependency of each test

Instead of passing file dependencies individually, assume that the
whole content of the unique test directory is a dependency. This
simplifies the test harness significantly, by making %T the directory
that contains everything required to run a test. This also removes the
need for the %{file_dependencies} substitution, which is removed by this
patch.

Furthermore, this patch also changes the harness to execute tests locally
inside %T, so as to avoid creating a separate directory for no purpose.




More information about the All-commits mailing list