diff options
| author | Daniel Dunbar <daniel@zuster.org> | 2009-04-07 22:36:33 +0000 |
|---|---|---|
| committer | Daniel Dunbar <daniel@zuster.org> | 2009-04-07 22:36:33 +0000 |
| commit | ea10ab668f8b6cdaa17723b37af32f1a7cb36e46 (patch) | |
| tree | 2e5208e75694f3e0d0f6927babb7e4ac4b0a1727 /clang/lib/CodeGen/CodeGenModule.cpp | |
| parent | 4aa25b79f941abd481266c5fb506354477a92ddf (diff) | |
| download | bcm5719-llvm-ea10ab668f8b6cdaa17723b37af32f1a7cb36e46.tar.gz bcm5719-llvm-ea10ab668f8b6cdaa17723b37af32f1a7cb36e46.zip | |
Visibility attributes should only be set on definition.
llvm-svn: 68561
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
| -rw-r--r-- | clang/lib/CodeGen/CodeGenModule.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index 1dd2c4d67f0..508d7ce2b6f 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -272,10 +272,12 @@ void CodeGenModule::SetGlobalValueAttributes(const Decl *D, // FIXME: Figure out the relative priority of the attribute, // -fvisibility, and private_extern. - if (const VisibilityAttr *attr = D->getAttr<VisibilityAttr>()) - setGlobalVisibility(GV, attr->getVisibility()); - else - setGlobalOptionVisibility(GV, getLangOptions().getVisibilityMode()); + if (ForDefinition) { + if (const VisibilityAttr *attr = D->getAttr<VisibilityAttr>()) + setGlobalVisibility(GV, attr->getVisibility()); + else + setGlobalOptionVisibility(GV, getLangOptions().getVisibilityMode()); + } if (const SectionAttr *SA = D->getAttr<SectionAttr>()) GV->setSection(SA->getName()); |
