diff options
author | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-01-13 02:14:39 +0000 |
---|---|---|
committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-01-13 02:14:39 +0000 |
commit | 90b748e0d1a86d998e204ba2fb1ee2602d0cc6e2 (patch) | |
tree | 83fc46b4fc83c6d2f1174db337fce0baba14a1c8 /clang/lib/Sema/SemaFixItUtils.cpp | |
parent | bc2f91820038c88b50b619327746b1957a19c5b9 (diff) | |
download | llvm-90b748e0d1a86d998e204ba2fb1ee2602d0cc6e2.zip llvm-90b748e0d1a86d998e204ba2fb1ee2602d0cc6e2.tar.gz llvm-90b748e0d1a86d998e204ba2fb1ee2602d0cc6e2.tar.bz2 |
Don't crash while trying to diagnose a function declared at block scope with an
incomplete return type.
llvm-svn: 148088
Diffstat (limited to 'clang/lib/Sema/SemaFixItUtils.cpp')
-rw-r--r-- | clang/lib/Sema/SemaFixItUtils.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaFixItUtils.cpp b/clang/lib/Sema/SemaFixItUtils.cpp index 1f17a9e..0f7530b 100644 --- a/clang/lib/Sema/SemaFixItUtils.cpp +++ b/clang/lib/Sema/SemaFixItUtils.cpp @@ -180,9 +180,11 @@ const char *Sema::getFixItZeroInitializerForType(QualType T) const { if (T->isScalarType()) return " = 0"; const CXXRecordDecl *RD = T->getAsCXXRecordDecl(); - if (LangOpts.CPlusPlus0x && RD && !RD->hasUserProvidedDefaultConstructor()) + if (!RD || !RD->hasDefinition()) + return 0; + if (LangOpts.CPlusPlus0x && !RD->hasUserProvidedDefaultConstructor()) return "{}"; - if (T->isAggregateType()) + if (RD->isAggregate()) return " = {}"; return 0; } |