[PATCH] D90281: [FileCheck] Report missing prefixes when more than one is provided.
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 28 23:39:17 PDT 2020
grimar added inline comments.
================
Comment at: llvm/test/FileCheck/allow-unused-prefixes.txt:1
+// RUN: not FileCheck --allow-unused-prefixes=false --check-prefixes=P1,P2 --input-file %s %s
+// RUN: FileCheck --allow-unused-prefixes=true --check-prefixes=P1,P2 --input-file %s %s
----------------
You probably need to test the error reported?
================
Comment at: llvm/test/FileCheck/allow-unused-prefixes.txt:2
+// RUN: not FileCheck --allow-unused-prefixes=false --check-prefixes=P1,P2 --input-file %s %s
+// RUN: FileCheck --allow-unused-prefixes=true --check-prefixes=P1,P2 --input-file %s %s
+
----------------
I think it also worth to add a test for the default value, i.e. when no `--allow-unused-prefixes` is present.
================
Comment at: llvm/test/FileCheck/allow-unused-prefixes.txt:6
+; P1: hello
\ No newline at end of file
----------------
No EOL.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90281/new/
https://reviews.llvm.org/D90281
More information about the llvm-commits
mailing list