[all-commits] [llvm/llvm-project] db8dad: [ELF][test] Change references of %T to %t.dir

Fangrui Song via All-commits all-commits at lists.llvm.org
Wed Oct 30 09:23:17 PDT 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: db8dad20b37f4befe054e586b40f8dd1eaeda842
      https://github.com/llvm/llvm-project/commit/db8dad20b37f4befe054e586b40f8dd1eaeda842
  Author: Fangrui Song <maskray at google.com>
  Date:   2019-10-30 (Wed, 30 Oct 2019)

  Changed paths:
    M lld/test/ELF/linkerscript/filename-spec.s
    M lld/test/ELF/linkerscript/linker-script-in-search-path.s
    M lld/test/ELF/linkerscript/linkerscript.s
    M lld/test/ELF/linkerscript/memory-include.test
    M lld/test/ELF/linkerscript/output-section-include.test
    M lld/test/ELF/linkerscript/section-include.test
    M lld/test/ELF/lto/save-temps.ll
    M lld/test/ELF/lto/thinlto-debug-fission.ll
    M lld/test/ELF/no-soname.s
    M lld/test/ELF/reproduce-backslash.s
    M lld/test/ELF/version-script-in-search-path.s

  Log Message:
  -----------
  [ELF][test] Change references of %T to %t.dir

Test files in the same directory share the same %T.  %T is easy to
misuse and cause race conditions (when running concurrently) so it has
been deprecated since D48842 (see docs/CommandGuide/lit.rst).

While here, add `rm -rf %t.dir` so that tests cannot depend on old files lying around.

Reviewed By: jhenderson, ruiu

Differential Revision: https://reviews.llvm.org/D69572




More information about the All-commits mailing list