[all-commits] [llvm/llvm-project] 871d65: [FileCheck] Report missing prefixes when more than...

Mircea Trofin via All-commits all-commits at lists.llvm.org
Fri Oct 30 12:39:52 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 871d658c9ceb391df34e43d8f7af596c0b8a36df
      https://github.com/llvm/llvm-project/commit/871d658c9ceb391df34e43d8f7af596c0b8a36df
  Author: Mircea Trofin <mtrofin at google.com>
  Date:   2020-10-30 (Fri, 30 Oct 2020)

  Changed paths:
    M llvm/include/llvm/FileCheck/FileCheck.h
    M llvm/lib/FileCheck/FileCheck.cpp
    A llvm/test/FileCheck/Inputs/one-check.txt
    A llvm/test/FileCheck/allow-unused-prefixes.txt
    M llvm/utils/FileCheck/FileCheck.cpp

  Log Message:
  -----------
  [FileCheck] Report missing prefixes when more than one is provided.

If more than a prefix is provided - e.g. --check-prefixes=CHECK,FOO - we
don't report if (say) FOO is never used. This may lead to a gap in our
test coverage.

This patch introduces a new option, --allow-unused-prefixes. It
currently is set to true, keeping today's behavior. After we explicitly
set it in tests where this behavior was actually intentional, we will
switch it to false by default.

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




More information about the All-commits mailing list