[PATCH] D78024: [FileCheck] - Fix the false positive when -implicit-check-not is used with an unknown -check-prefix.
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 15 06:32:37 PDT 2020
grimar marked an inline comment as done.
grimar added inline comments.
================
Comment at: llvm/lib/Support/FileCheck.cpp:1393-1394
+ // Add an EOF pattern for any trailing CHECK-DAG/-NOTs, and use the first
+ // prefix as a filler for the error message.
+ if (!DagNotMatches.empty()) {
----------------
jdenny wrote:
> Your original patch mentioned -implicit-check-not here. I thought that was helpful. Is there a reason to drop it?
Yeah, I've dropped it because it is unrelated to the latest version of the patch, as now this piece of the code is unchanged, it just moved.
I'll commit a NFC change for this comment separatelly then.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78024/new/
https://reviews.llvm.org/D78024
More information about the llvm-commits
mailing list