[all-commits] [llvm/llvm-project] e9a3d1: [FileCheck] Move FileCheck implementation out of L...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Mon Aug 31 02:25:17 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e9a3d1a401b07cbf7b11695637f1b549782a26cd
      https://github.com/llvm/llvm-project/commit/e9a3d1a401b07cbf7b11695637f1b549782a26cd
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2020-08-31 (Mon, 31 Aug 2020)

  Changed paths:
    A llvm/include/llvm/FileCheck/FileCheck.h
    R llvm/include/llvm/Support/FileCheck.h
    M llvm/include/llvm/module.modulemap
    M llvm/lib/CMakeLists.txt
    A llvm/lib/FileCheck/CMakeLists.txt
    A llvm/lib/FileCheck/FileCheck.cpp
    A llvm/lib/FileCheck/FileCheckImpl.h
    M llvm/lib/Support/CMakeLists.txt
    R llvm/lib/Support/FileCheck.cpp
    R llvm/lib/Support/FileCheckImpl.h
    M llvm/unittests/CMakeLists.txt
    M llvm/unittests/CodeGen/GlobalISel/CMakeLists.txt
    M llvm/unittests/CodeGen/GlobalISel/GISelMITest.h
    A llvm/unittests/FileCheck/CMakeLists.txt
    A llvm/unittests/FileCheck/FileCheckTest.cpp
    M llvm/unittests/Support/CMakeLists.txt
    R llvm/unittests/Support/FileCheckTest.cpp
    M llvm/utils/FileCheck/CMakeLists.txt
    M llvm/utils/FileCheck/FileCheck.cpp
    A llvm/utils/gn/secondary/llvm/lib/FileCheck/BUILD.gn
    M llvm/utils/gn/secondary/llvm/lib/Support/BUILD.gn
    M llvm/utils/gn/secondary/llvm/unittests/BUILD.gn
    M llvm/utils/gn/secondary/llvm/unittests/CodeGen/GlobalISel/BUILD.gn
    A llvm/utils/gn/secondary/llvm/unittests/FileCheck/BUILD.gn
    M llvm/utils/gn/secondary/llvm/unittests/Support/BUILD.gn

  Log Message:
  -----------
  [FileCheck] Move FileCheck implementation out of LLVMSupport into its own library

The actual FileCheck logic seems to be implemented in LLVMSupport. I don't see a
good reason for having FileCheck implemented there as it has a very specific use
while LLVMSupport is a dependency of pretty much every LLVM tool there is. In
fact, the only use of FileCheck I could find (outside the FileCheck tool and the
FileCheck unit test) is a single call in GISelMITest.h.

This moves the FileCheck logic to its own LLVMFileCheck library. This way only
FileCheck and the GlobalISelTests now have a dependency on this code.

Reviewed By: jhenderson

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




More information about the All-commits mailing list