diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2012-04-18 00:14:58 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2012-04-18 00:14:58 +0000 |
| commit | 12b785e8bb696fed343a4aa4be0d14bab669d0ab (patch) | |
| tree | abca5baf886e5f19b9a38ecc55cf68c1bbfaacfb /clang/lib/AST/Decl.cpp | |
| parent | 3a940250bf3cbcdb274bd260617ba45dc41785ac (diff) | |
| download | bcm5719-llvm-12b785e8bb696fed343a4aa4be0d14bab669d0ab.tar.gz bcm5719-llvm-12b785e8bb696fed343a4aa4be0d14bab669d0ab.zip | |
Calling setVisibility directly only makes (some) sense when the visibility is
explicit.
llvm-svn: 154969
Diffstat (limited to 'clang/lib/AST/Decl.cpp')
| -rw-r--r-- | clang/lib/AST/Decl.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/clang/lib/AST/Decl.cpp b/clang/lib/AST/Decl.cpp index 07098fdb3a4..53032bc64d6 100644 --- a/clang/lib/AST/Decl.cpp +++ b/clang/lib/AST/Decl.cpp @@ -785,7 +785,7 @@ static LinkageInfo getLVForDecl(const NamedDecl *D, LVFlags Flags) { LinkageInfo LV; if (Flags.ConsiderVisibilityAttributes) { if (llvm::Optional<Visibility> Vis = Function->getExplicitVisibility()) - LV.setVisibility(*Vis); + LV.setVisibility(*Vis, true); } if (const FunctionDecl *Prev = Function->getPreviousDecl()) { @@ -806,10 +806,10 @@ static LinkageInfo getLVForDecl(const NamedDecl *D, LVFlags Flags) { LinkageInfo LV; if (Var->getStorageClass() == SC_PrivateExtern) - LV.setVisibility(HiddenVisibility); + LV.setVisibility(HiddenVisibility, true); else if (Flags.ConsiderVisibilityAttributes) { if (llvm::Optional<Visibility> Vis = Var->getExplicitVisibility()) - LV.setVisibility(*Vis); + LV.setVisibility(*Vis, true); } if (const VarDecl *Prev = Var->getPreviousDecl()) { |
