[PATCH] D60170: [llvm-objcopy] [llvm-symbolizer] Fix failing tests

[Don Hinton via Phabricator via llvm-commits]

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


================
Comment at: llvm/test/tools/llvm-symbolizer/ignore-undefined-symbols.s:3
 # RUN: llvm-mc --filetype=obj --triple=x86_64-pc-linux %s -o %t.o -g
-# RUN: llvm-symbolizer --obj=%t.o 0 | FileCheck %s --implicit-check-not=bar
+# RUN: llvm-symbolizer --obj=%t.o 0 | FileCheck %s --implicit-check-not="^bar$"
 
----------------
hintonda wrote:
> jhenderson wrote:
> > Is implicit-check-not a regex pattern or just a literal check (I think it's a literal check)? Does this still fail when bar appears instead of foo?
> Yes, you're correct, it's a literal check.  I can reproduce by using my login -- which obviously isn't a symbol:
> 
> ```
> $ bin/llvm-symbolizer --obj=x.o 0 | bin/FileCheck /Users/dhinton/projects/llvm_project/monorepo/llvm-project/llvm/test/tools/llvm-symbolizer/ignore-undefined-symbols.s --implicit-check-not=dhinton
> command line:1:22: error: CHECK-NOT: excluded string found in input
> -implicit-check-not='dhinton'
>                      ^
> <stdin>:2:8: note: found here
> /Users/dhinton/projects/llvm_project/monorepo/build/Debug/../../llvm-project/llvm/test/tools/llvm-symbolizer/ignore-undefined-symbols.s:12:0
>        ^~~~~~~
> ```
> 
> I'll see if I can come up with a better solution.
Okay, prepending `{{.*}}` to `ignore-undefined-symbols` below fixes the problem.  Will update shortly.  Thanks for the feedback.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60170