[PATCH] D95249: [NFC] Disallow unused prefixes in clang/test/Analysis
This revision was automatically updated to reflect the committed changes. Closed by commit rG91b61abafb5a: [NFC] Disallow unused prefixes in clang/test/Analysis (authored by mtrofin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95249/new/ https://reviews.llvm.org/D95249 Files: clang/test/Analysis/auto-obj-dtors-cfg-output.cpp clang/test/Analysis/cfg-rich-constructors.cpp clang/test/Analysis/cfg-rich-constructors.mm clang/test/Analysis/cfg.c clang/test/Analysis/exploded-graph-rewriter/trimmers.dot clang/test/Analysis/lit.local.cfg Index: clang/test/Analysis/lit.local.cfg === --- clang/test/Analysis/lit.local.cfg +++ clang/test/Analysis/lit.local.cfg @@ -1,5 +1,5 @@ # -*- Python -*- vim: set ft=python ts=4 sw=4 expandtab tw=79: - +from lit.llvm.subst import ToolSubst import site # Load the custom analyzer test format, which runs the test again with Z3 if it @@ -26,3 +26,9 @@ if not config.root.clang_staticanalyzer: config.unsupported = True + +fc = ToolSubst('FileCheck', unresolved='fatal') +# Insert this first. Then, we'll first update the blank FileCheck command; then, +# the default substitution of FileCheck will replace it to its full path. +config.substitutions.insert(0, (fc.regex, +'FileCheck --allow-unused-prefixes=false')) Index: clang/test/Analysis/exploded-graph-rewriter/trimmers.dot === --- clang/test/Analysis/exploded-graph-rewriter/trimmers.dot +++ clang/test/Analysis/exploded-graph-rewriter/trimmers.dot @@ -1,17 +1,17 @@ // RUN: %exploded_graph_rewriter %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,FOUR // RUN: %exploded_graph_rewriter --to=0x2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 2,3 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 4 %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter --to 4 -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,FOUR Node0x1 [shape=record,label= "{{ "state_id": 0, "program_state": null, "program_points": [ Index: clang/test/Analysis/cfg.c === --- clang/test/Analysis/cfg.c +++ clang/test/Analysis/cfg.c @@ -1,5 +1,5 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -fheinous-gnu-extensions %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,WARNINGS %s +// RUN: FileCheck --input-file=%t --check-prefix=CHECK %s // This file is the C version of cfg.cpp. // Tests that are C-specific should go into this file. Index: clang/test/Analysis/cfg-rich-constructors.mm === --- clang/test/Analysis/cfg-rich-constructors.mm +++ clang/test/Analysis/cfg-rich-constructors.mm @@ -1,11 +1,11 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,ELIDE,CXX11-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-ELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,ELIDE,CXX17-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,NOELIDE,CXX11-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-NOELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,NOELIDE,CXX17-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s class D { public: Index: clang/test/Analysis/cfg-rich-constructors.cpp === ---
[PATCH] D95249: [NFC] Disallow unused prefixes in clang/test/Analysis
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thanks!! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95249/new/ https://reviews.llvm.org/D95249 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D95249: [NFC] Disallow unused prefixes in clang/test/Analysis
mtrofin updated this revision to Diff 319024. mtrofin added a comment. fixed trimmers.dot Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95249/new/ https://reviews.llvm.org/D95249 Files: clang/test/Analysis/auto-obj-dtors-cfg-output.cpp clang/test/Analysis/cfg-rich-constructors.cpp clang/test/Analysis/cfg-rich-constructors.mm clang/test/Analysis/cfg.c clang/test/Analysis/exploded-graph-rewriter/trimmers.dot clang/test/Analysis/lit.local.cfg Index: clang/test/Analysis/lit.local.cfg === --- clang/test/Analysis/lit.local.cfg +++ clang/test/Analysis/lit.local.cfg @@ -1,5 +1,5 @@ # -*- Python -*- vim: set ft=python ts=4 sw=4 expandtab tw=79: - +from lit.llvm.subst import ToolSubst import site # Load the custom analyzer test format, which runs the test again with Z3 if it @@ -26,3 +26,9 @@ if not config.root.clang_staticanalyzer: config.unsupported = True + +fc = ToolSubst('FileCheck', unresolved='fatal') +# Insert this first. Then, we'll first update the blank FileCheck command; then, +# the default substitution of FileCheck will replace it to its full path. +config.substitutions.insert(0, (fc.regex, +'FileCheck --allow-unused-prefixes=false')) Index: clang/test/Analysis/exploded-graph-rewriter/trimmers.dot === --- clang/test/Analysis/exploded-graph-rewriter/trimmers.dot +++ clang/test/Analysis/exploded-graph-rewriter/trimmers.dot @@ -1,17 +1,17 @@ // RUN: %exploded_graph_rewriter %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,FOUR // RUN: %exploded_graph_rewriter --to=0x2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 2,3 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,NOTFOUR // RUN: %exploded_graph_rewriter --to 4 %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter --to 4 -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTTHREE,FOUR Node0x1 [shape=record,label= "{{ "state_id": 0, "program_state": null, "program_points": [ Index: clang/test/Analysis/cfg.c === --- clang/test/Analysis/cfg.c +++ clang/test/Analysis/cfg.c @@ -1,5 +1,5 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -fheinous-gnu-extensions %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,WARNINGS %s +// RUN: FileCheck --input-file=%t --check-prefix=CHECK %s // This file is the C version of cfg.cpp. // Tests that are C-specific should go into this file. Index: clang/test/Analysis/cfg-rich-constructors.mm === --- clang/test/Analysis/cfg-rich-constructors.mm +++ clang/test/Analysis/cfg-rich-constructors.mm @@ -1,11 +1,11 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,ELIDE,CXX11-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-ELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,ELIDE,CXX17-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,NOELIDE,CXX11-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-NOELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,NOELIDE,CXX17-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s class D { public: Index: clang/test/Analysis/cfg-rich-constructors.cpp === --- clang/test/Analysis/cfg-rich-constructors.cpp +++ clang/test/Analysis/cfg-rich-constructors.cpp @@ -1,11 +1,11 @@
[PATCH] D95249: [NFC] Disallow unused prefixes in clang/test/Analysis
NoQ added a comment. I added those for consistency but i think your effort is worth it so let's ditch them. Comment at: clang/test/Analysis/exploded-graph-rewriter/trimmers.dot:4 // RUN: %exploded_graph_rewriter -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR // RUN: %exploded_graph_rewriter --to=0x2 %s \ Uh-oh, these look like typos as there are lines for `NOTTHREE` and `NOTFOUR`. Looks like your effort has paid off and found a real bug in tests! The tests seem to be passing just fine after i change to `NOTTHREE` and `NOTFOUR` respectively. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95249/new/ https://reviews.llvm.org/D95249 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D95249: [NFC] Disallow unused prefixes in clang/test/Analysis
mtrofin created this revision. mtrofin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D95249 Files: clang/test/Analysis/auto-obj-dtors-cfg-output.cpp clang/test/Analysis/cfg-rich-constructors.cpp clang/test/Analysis/cfg-rich-constructors.mm clang/test/Analysis/cfg.c clang/test/Analysis/exploded-graph-rewriter/trimmers.dot clang/test/Analysis/lit.local.cfg Index: clang/test/Analysis/lit.local.cfg === --- clang/test/Analysis/lit.local.cfg +++ clang/test/Analysis/lit.local.cfg @@ -1,5 +1,5 @@ # -*- Python -*- vim: set ft=python ts=4 sw=4 expandtab tw=79: - +from lit.llvm.subst import ToolSubst import site # Load the custom analyzer test format, which runs the test again with Z3 if it @@ -26,3 +26,9 @@ if not config.root.clang_staticanalyzer: config.unsupported = True + +fc = ToolSubst('FileCheck', unresolved='fatal') +# Insert this first. Then, we'll first update the blank FileCheck command; then, +# the default substitution of FileCheck will replace it to its full path. +config.substitutions.insert(0, (fc.regex, +'FileCheck --allow-unused-prefixes=false')) Index: clang/test/Analysis/exploded-graph-rewriter/trimmers.dot === --- clang/test/Analysis/exploded-graph-rewriter/trimmers.dot +++ clang/test/Analysis/exploded-graph-rewriter/trimmers.dot @@ -1,17 +1,17 @@ // RUN: %exploded_graph_rewriter %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,FOUR // RUN: %exploded_graph_rewriter --to=0x2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO // RUN: %exploded_graph_rewriter --to 2 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO // RUN: %exploded_graph_rewriter --to 2,3 %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,NOFOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE // RUN: %exploded_graph_rewriter --to 4 %s \ // RUN: | FileCheck %s -check-prefixes=ONE,TWO,THREE,FOUR // RUN: %exploded_graph_rewriter --to 4 -s %s \ -// RUN: | FileCheck %s -check-prefixes=ONE,TWO,NOTHREE,FOUR +// RUN: | FileCheck %s -check-prefixes=ONE,TWO,FOUR Node0x1 [shape=record,label= "{{ "state_id": 0, "program_state": null, "program_points": [ Index: clang/test/Analysis/cfg.c === --- clang/test/Analysis/cfg.c +++ clang/test/Analysis/cfg.c @@ -1,5 +1,5 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -fheinous-gnu-extensions %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,WARNINGS %s +// RUN: FileCheck --input-file=%t --check-prefix=CHECK %s // This file is the C version of cfg.cpp. // Tests that are C-specific should go into this file. Index: clang/test/Analysis/cfg-rich-constructors.mm === --- clang/test/Analysis/cfg-rich-constructors.mm +++ clang/test/Analysis/cfg-rich-constructors.mm @@ -1,11 +1,11 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,ELIDE,CXX11-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-ELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,ELIDE,CXX17-ELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++11 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,NOELIDE,CXX11-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX11,CXX11-NOELIDE %s // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple x86_64-apple-darwin12 -std=c++17 -w -analyzer-config elide-constructors=false %s > %t 2>&1 -// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17,NOELIDE,CXX17-NOELIDE %s +// RUN: FileCheck --input-file=%t -check-prefixes=CHECK,CXX17 %s class D { public: Index: clang/test/Analysis/cfg-rich-constructors.cpp === --- clang/test/Analysis/cfg-rich-constructors.cpp +++ clang/test/Analysis/cfg-rich-constructors.cpp @@ -1,11 +1,11 @@ // RUN: %clang_analyze_cc1 -analyzer-checker=debug.DumpCFG -triple