[PATCH] D61727: [llvm-objcopy] Add additional testing for various cases

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 5 06:55:50 PDT 2020


thakis added inline comments.
Herald added a subscriber: abrachet.


================
Comment at: llvm/trunk/test/tools/llvm-objcopy/ELF/shared-strtab-shstrtab.s:63
+# RUN: llvm-readobj --section-headers --symbols %t.remove-symbol \
+# RUN:   | FileCheck %s --check-prefix=SYMBOL-REMOVE --implicit-check-not=.shstrtab --implicit-check-not=foo
+
----------------
This makes the test fail if "foo" is a substring of the path of your build dir. Could you make this more robust against accidentally matching the directory name?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61727





More information about the llvm-commits mailing list