diff options
| author | John McCall <rjmccall@apple.com> | 2010-12-10 02:59:44 +0000 |
|---|---|---|
| committer | John McCall <rjmccall@apple.com> | 2010-12-10 02:59:44 +0000 |
| commit | 90f1450c109fbbd333001165bbd986061f7c4513 (patch) | |
| tree | eecf97a3097543529850ba046c5b0440d645f668 /lib/Sema/SemaDeclCXX.cpp | |
| parent | f0d9d8b79389e398ea0c44cde9d7b9020e0ccb65 (diff) | |
Treat visibility on an enclosing namespace as a non-explicit source of
visibility. Fixes PR8713.
I've disabled a test which was testing that you can #pragma pop visibility
to get out of a namespace's visibility attribute. We should probably just
diagnose that as an error unless it's instrumental to someone's system
headers.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@121459 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaDeclCXX.cpp')
| -rw-r--r-- | lib/Sema/SemaDeclCXX.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/Sema/SemaDeclCXX.cpp b/lib/Sema/SemaDeclCXX.cpp index bf9e935981..b5e6321276 100644 --- a/lib/Sema/SemaDeclCXX.cpp +++ b/lib/Sema/SemaDeclCXX.cpp @@ -3194,8 +3194,8 @@ Decl *Sema::ActOnStartNamespaceDef(Scope *NamespcScope, ProcessDeclAttributeList(DeclRegionScope, Namespc, AttrList); - if (const VisibilityAttr *attr = Namespc->getAttr<VisibilityAttr>()) - PushVisibilityAttr(attr); + if (const VisibilityAttr *Attr = Namespc->getAttr<VisibilityAttr>()) + PushNamespaceVisibilityAttr(Attr); if (II) { // C++ [namespace.def]p2: |