[PATCH] D67069: [llvm-strings][test] Merge two closely-related tests
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 2 04:28:47 PDT 2019
jhenderson marked an inline comment as done.
jhenderson added inline comments.
================
Comment at: llvm/test/tools/llvm-strings/eof.test:11
+RUN: echo -n abc | llvm-strings - | FileCheck --allow-empty %s --check-prefix=NOPRINT
+NOPRINT-NOT: abc
----------------
grimar wrote:
> `NOPRINT-NOT: {{.}}` would probably be a bit better?
Yes, I was thinking about that when I did this. I'll update before committing.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67069/new/
https://reviews.llvm.org/D67069
More information about the llvm-commits
mailing list