summaryrefslogtreecommitdiffstats
path: root/clang/lib/AST
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2009-12-08 03:56:49 +0000
committerEli Friedman <eli.friedman@gmail.com>2009-12-08 03:56:49 +0000
commitf2c79b6b9ca75424ceeb246c91916728ee3a26fd (patch)
treed025836729e4c763dcdef2388ab1369242309773 /clang/lib/AST
parent4569f69558efad0ad6dbc68a58f1b4955b67caae (diff)
downloadbcm5719-llvm-f2c79b6b9ca75424ceeb246c91916728ee3a26fd.tar.gz
bcm5719-llvm-f2c79b6b9ca75424ceeb246c91916728ee3a26fd.zip
Misc key function fixes.
llvm-svn: 90831
Diffstat (limited to 'clang/lib/AST')
-rw-r--r--clang/lib/AST/RecordLayoutBuilder.cpp11
1 files changed, 10 insertions, 1 deletions
diff --git a/clang/lib/AST/RecordLayoutBuilder.cpp b/clang/lib/AST/RecordLayoutBuilder.cpp
index f7da8903504..c914f3f82e8 100644
--- a/clang/lib/AST/RecordLayoutBuilder.cpp
+++ b/clang/lib/AST/RecordLayoutBuilder.cpp
@@ -724,7 +724,13 @@ ASTRecordLayoutBuilder::ComputeKeyFunction(const CXXRecordDecl *RD) {
// function.
if (RD->getTemplateSpecializationKind() != TSK_Undeclared)
return 0;
-
+
+ // A class inside an anonymous namespace doesn't have a key function. (Or
+ // at least, there's no point to assigning a key function to such a class;
+ // this doesn't affect the ABI.)
+ if (RD->isInAnonymousNamespace())
+ return 0;
+
for (CXXRecordDecl::method_iterator I = RD->method_begin(),
E = RD->method_end(); I != E; ++I) {
const CXXMethodDecl *MD = *I;
@@ -734,6 +740,9 @@ ASTRecordLayoutBuilder::ComputeKeyFunction(const CXXRecordDecl *RD) {
if (MD->isPure())
continue;
+
+ if (MD->isInlineSpecified())
+ continue;
// Ignore implicit member functions, they are always marked as inline, but
// they don't have a body until they're defined.
OpenPOWER on IntegriCloud