diff options
| author | Lang Hames <lhames@gmail.com> | 2015-01-22 22:04:47 +0000 |
|---|---|---|
| committer | Lang Hames <lhames@gmail.com> | 2015-01-22 22:04:47 +0000 |
| commit | bd4ccc89e61b77c9a72a0627026055a0befb91a1 (patch) | |
| tree | b103c644808443cd17bf03968691efb500750d55 | |
| parent | df8b223deae9e2a4a127d379a9ce2701b68c8414 (diff) | |
| download | bcm5719-llvm-bd4ccc89e61b77c9a72a0627026055a0befb91a1.tar.gz bcm5719-llvm-bd4ccc89e61b77c9a72a0627026055a0befb91a1.zip | |
[Object] Fix a bug in a condition introduced in r226217 - visibility can't be
both hidden and default.
Bug found by inspection by Rafael Espindola. No test: As discussed in the commit
message for r226217 we don't have a good way to test this yet.
llvm-svn: 226869
| -rw-r--r-- | llvm/include/llvm/Object/ELFObjectFile.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/include/llvm/Object/ELFObjectFile.h b/llvm/include/llvm/Object/ELFObjectFile.h index 07f9542263b..125fdd17743 100644 --- a/llvm/include/llvm/Object/ELFObjectFile.h +++ b/llvm/include/llvm/Object/ELFObjectFile.h @@ -185,7 +185,7 @@ protected: // visibility is either DEFAULT or PROTECTED. All other symbols are not // exported. if ((Binding == ELF::STB_GLOBAL || Binding == ELF::STB_WEAK) && - (Visibility == ELF::STV_DEFAULT && Visibility == ELF::STV_PROTECTED)) + (Visibility == ELF::STV_DEFAULT || Visibility == ELF::STV_PROTECTED)) return true; return false; |
