aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--include/clang/AST/DeclBase.h15
-rw-r--r--lib/AST/DeclBase.cpp9
-rw-r--r--lib/Sema/SemaDecl.cpp6
-rw-r--r--test/SemaCXX/nested-name-spec.cpp7
4 files changed, 34 insertions, 3 deletions
diff --git a/include/clang/AST/DeclBase.h b/include/clang/AST/DeclBase.h
index bfc7607b74..12ed5ff174 100644
--- a/include/clang/AST/DeclBase.h
+++ b/include/clang/AST/DeclBase.h
@@ -300,6 +300,21 @@ public:
const_cast<const DeclContext*>(this)->getParent());
}
+ /// getLexicalParent - Returns the containing lexical DeclContext. May be
+ /// different from getParent, e.g.:
+ ///
+ /// namespace A {
+ /// struct S;
+ /// }
+ /// struct A::S {}; // getParent() == namespace 'A'
+ /// // getLexicalParent() == translation unit
+ ///
+ const DeclContext *getLexicalParent() const;
+ DeclContext *getLexicalParent() {
+ return const_cast<DeclContext*>(
+ const_cast<const DeclContext*>(this)->getLexicalParent());
+ }
+
bool isFunctionOrMethod() const {
switch (DeclKind) {
case Decl::Block:
diff --git a/lib/AST/DeclBase.cpp b/lib/AST/DeclBase.cpp
index 88d9a20855..c0655cf9ce 100644
--- a/lib/AST/DeclBase.cpp
+++ b/lib/AST/DeclBase.cpp
@@ -361,3 +361,12 @@ const DeclContext *DeclContext::getParent() const {
else
return NULL;
}
+
+const DeclContext *DeclContext::getLexicalParent() const {
+ if (const ScopedDecl *SD = dyn_cast<ScopedDecl>(this))
+ return SD->getLexicalDeclContext();
+ else if (const BlockDecl *BD = dyn_cast<BlockDecl>(this))
+ return BD->getParentContext();
+ else
+ return NULL;
+}
diff --git a/lib/Sema/SemaDecl.cpp b/lib/Sema/SemaDecl.cpp
index d6d9845af7..941f40fa52 100644
--- a/lib/Sema/SemaDecl.cpp
+++ b/lib/Sema/SemaDecl.cpp
@@ -53,10 +53,10 @@ DeclContext *Sema::getContainingDC(DeclContext *DC) {
// A C++ inline method is parsed *after* the topmost class it was declared in
// is fully parsed (it's "complete").
// The parsing of a C++ inline method happens at the declaration context of
- // the topmost (non-nested) class it is declared in.
+ // the topmost (non-nested) class it is lexically declared in.
assert(isa<CXXRecordDecl>(MD->getParent()) && "C++ method not in Record.");
DC = MD->getParent();
- while (CXXRecordDecl *RD = dyn_cast<CXXRecordDecl>(DC->getParent()))
+ while (CXXRecordDecl *RD = dyn_cast<CXXRecordDecl>(DC->getLexicalParent()))
DC = RD;
// Return the declaration context of the topmost class the inline method is
@@ -70,7 +70,7 @@ DeclContext *Sema::getContainingDC(DeclContext *DC) {
if (ScopedDecl *SD = dyn_cast<ScopedDecl>(DC))
return SD->getLexicalDeclContext();
- return DC->getParent();
+ return DC->getLexicalParent();
}
void Sema::PushDeclContext(DeclContext *DC) {
diff --git a/test/SemaCXX/nested-name-spec.cpp b/test/SemaCXX/nested-name-spec.cpp
index b22d8dc3d4..18cb3b418a 100644
--- a/test/SemaCXX/nested-name-spec.cpp
+++ b/test/SemaCXX/nested-name-spec.cpp
@@ -45,12 +45,19 @@ struct A::undef; // expected-error {{'undef' does not name a tag member in the s
namespace A2 {
typedef int INT;
struct RC;
+ struct CC {
+ struct NC;
+ };
}
struct A2::RC {
INT x;
};
+struct A2::CC::NC {
+ void m() {}
+};
+
void f3() {
N::x = 0; // expected-error {{use of undeclared identifier 'N'}}
int N;