commit 90b748e0d1a86d998e204ba2fb1ee2602d0cc6e2
author Richard Smith <richard-llvm@metafoo.co.uk> Fri Jan 13 02:14:39 2012 +0000
committer Richard Smith <richard-llvm@metafoo.co.uk> Fri Jan 13 02:14:39 2012 +0000
tree 83fc46b4fc83c6d2f1174db337fce0baba14a1c8
parent bc2f91820038c88b50b619327746b1957a19c5b9

Don't crash while trying to diagnose a function declared at block scope with an
incomplete return type.

llvm-svn: 148088

clang/lib/Sema/SemaDecl.cpp

diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index 4dcf1db..20309ee 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -4945,7 +4945,8 @@
         // Empty parens mean value-initialization, and no parens mean default
         // initialization. These are equivalent if the default constructor is
         // user-provided, or if zero-initialization is a no-op.
-        if (RD && (RD->isEmpty() || RD->hasUserProvidedDefaultConstructor()))
+        if (RD && RD->hasDefinition() &&
+            (RD->isEmpty() || RD->hasUserProvidedDefaultConstructor()))
           Diag(C.Loc, diag::note_empty_parens_default_ctor)
             << FixItHint::CreateRemoval(ParenRange);
         else if (const char *Init = getFixItZeroInitializerForType(T))