[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