[llvm] [llvm-objcopy] Add llvm-objcopy option --set-visibility-sym (PR #80872)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 01:15:33 PST 2024


================
@@ -0,0 +1,43 @@
+
+# RUN: yaml2obj %s -o %t.o
+# RUN: echo 'foo' > %t.symbols
+
+# Check if passing an invalid visibility type generates an error message
+# RUN: not llvm-objcopy %t.o --set-symbols-visibility=%t.symbols=invalid-type --regex 2>&1 | FileCheck %s --check-prefix=TYPE
----------------
jh7370 wrote:

Nit: we tend to break long lines like this over two, using the style in the suggested edit. Same applies elsewhere.
```suggestion
# RUN: not llvm-objcopy %t.o --set-symbols-visibility=%t.symbols=invalid-type --regex 2>&1 | \
# RUN:   FileCheck %s --check-prefix=TYPE
```

https://github.com/llvm/llvm-project/pull/80872


More information about the llvm-commits mailing list