[llvm] [llvm-objcopy] Add llvm-objcopy option --set-visibility-sym (PR #80872)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 8 00:38:57 PST 2024
================
@@ -290,6 +290,9 @@ static Error updateAndRemoveSymbols(const CommonConfig &Config,
return Error::success();
Obj.SymbolTable->updateSymbols([&](Symbol &Sym) {
+ if (ELFConfig.SymbolsToSetVisibility.matches(Sym.Name) &&
+ Sym.getShndx() != SHN_UNDEF)
----------------
jh7370 wrote:
I don't think you have any testing showing that SHN_UNDEF symbols aren't modified?
https://github.com/llvm/llvm-project/pull/80872
More information about the llvm-commits
mailing list