[PATCH] D57760: Fix defines.txt test failure on Windows targets

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 5 09:31:43 PST 2019


jhenderson added inline comments.


================
Comment at: llvm/test/FileCheck/defines.txt:27
 
-; ERRCLIEQ2: FileCheck{{[^:]*}}: for the -D option: requires a value!
+; ERRCLIEQ2: [fF]ile[cC]heck{{[^:]*}}: for the -D option: requires a value!
 
----------------
Maybe we just don't need to test that "FileCheck" appears in the error message at all? I'm not really convinced it belongs in the error message anyway, but that's a separate issue.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57760/new/

https://reviews.llvm.org/D57760





More information about the llvm-commits mailing list