diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2014-02-06 00:54:53 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2014-02-06 00:54:53 +0000 |
| commit | 12f04984f83ffc43d90d5eb058c9383047de1363 (patch) | |
| tree | 08b33c96c4ac8d639a13fcf886f229e35efaeca0 /llvm/lib/MC/ELFObjectWriter.cpp | |
| parent | 9724752f4b8864c599feffe11849075d83f26953 (diff) | |
| download | bcm5719-llvm-12f04984f83ffc43d90d5eb058c9383047de1363.tar.gz bcm5719-llvm-12f04984f83ffc43d90d5eb058c9383047de1363.zip | |
doesSectionRequireSymbols is meaningless on ELF, remove.
This is a nop. doesSectionRequireSymbols is only used from
isSymbolLinkerVisible. isSymbolLinkerVisible only use from ELF was in
if (!Asm.isSymbolLinkerVisible(Symbol) && !Symbol.isUndefined())
return false;
if (Symbol.isTemporary())
return false;
If the symbol is a temporary this code returns false and it is irrelevant if
we take the first if or not. If the symbol is not a temporary,
Asm.isSymbolLinkerVisible returns true without ever calling
doesSectionRequireSymbols.
This was an horrible leftover from when support for ELF was first added.
llvm-svn: 200894
Diffstat (limited to 'llvm/lib/MC/ELFObjectWriter.cpp')
| -rw-r--r-- | llvm/lib/MC/ELFObjectWriter.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/llvm/lib/MC/ELFObjectWriter.cpp b/llvm/lib/MC/ELFObjectWriter.cpp index f6dbfced540..8b2ab02bec5 100644 --- a/llvm/lib/MC/ELFObjectWriter.cpp +++ b/llvm/lib/MC/ELFObjectWriter.cpp @@ -848,9 +848,6 @@ bool ELFObjectWriter::isInSymtab(const MCAssembler &Asm, if (!Symbol.isVariable() && Symbol.isUndefined() && !IsGlobal) return false; - if (!Asm.isSymbolLinkerVisible(Symbol) && !Symbol.isUndefined()) - return false; - if (Symbol.isTemporary()) return false; |
