[PATCH] D153219: [symbolizer] Exit early if input file is absent

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 05:05:41 PDT 2023


sepavloff marked an inline comment as done.
sepavloff added inline comments.


================
Comment at: llvm/test/tools/llvm-symbolizer/pdb/missing_pdb.test:7
+CHECK: llvm-symbolizer{{.*}}: error: '{{.*}}missing_pdb.pdb': [[MSG]]
+CHECK-NOT: error reading file
+CHECK-NOT: ??
----------------
jhenderson wrote:
> Rather than checking that some arbitrary error message is not emitted, better would be to make sure nothing is printed beyond what you expect, with `--implicit-check-not={{.}}`.
> 
> (This would have the added benefit of not needing the `count 0` call, if you wanted, although I think @MaskRay's suggestion is cleaner for checking that stdout is empty.)
`--implicit-check-not` does not work, because input is empty and `FileCheck` complains, so used the method proposed by @MaskRay .


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153219



More information about the llvm-commits mailing list