r256511 - Do not crash if class is defined in wrong scope.

[Serge Pavlov via cfe-commits]

Author: sepavloff
Date: Mon Dec 28 13:40:14 2015
New Revision: 256511

URL: http://llvm.org/viewvc/llvm-project?rev=256511&view=rev
Log:
Do not crash if class is defined in wrong scope.

This patch fixes PR16677. The latter represents the case when due to
misprinted character class definition occurs in the scope of template
arguments. Base class of this class depends on the template parameter in the
same scope and cannot be resolved, it causes crash. Right behavior is to
make semantic processing even if the definition is wrong, as the code
that emits appropriate message is called after the processing.

Added:
    cfe/trunk/test/SemaCXX/PR16677.cpp
Modified:
    cfe/trunk/lib/Sema/SemaDeclCXX.cpp

Modified: cfe/trunk/lib/Sema/SemaDeclCXX.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclCXX.cpp?rev=256511&r1=256510&r2=256511&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDeclCXX.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDeclCXX.cpp Mon Dec 28 13:40:14 2015
@@ -5626,7 +5626,9 @@ bool SpecialMemberDeletionInfo::shouldDe
 /// having a particular direct or virtual base class.
 bool SpecialMemberDeletionInfo::shouldDeleteForBase(CXXBaseSpecifier *Base) {
   CXXRecordDecl *BaseClass = Base->getType()->getAsCXXRecordDecl();
-  return shouldDeleteForClassSubobject(BaseClass, Base, 0);
+  // If program is correct, BaseClass cannot be null, but if it is, the error
+  // must be reported elsewhere.
+  return BaseClass && shouldDeleteForClassSubobject(BaseClass, Base, 0);
 }
 
 /// Check whether we should delete a special member function due to the class

Added: cfe/trunk/test/SemaCXX/PR16677.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/PR16677.cpp?rev=256511&view=auto
==============================================================================
--- cfe/trunk/test/SemaCXX/PR16677.cpp (added)
+++ cfe/trunk/test/SemaCXX/PR16677.cpp Mon Dec 28 13:40:14 2015
@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 -fsyntax-only -std=c++11 -verify %s
+
+class Class_With_Destructor {
+  ~Class_With_Destructor() { }
+};
+
+template <class T>
+class Base { };
+
+template<class T,  // Should be angle bracket instead of comma
+class Derived : public Base<T> { // expected-error{{'Derived' cannot be defined in a type specifier}}
+  Class_With_Destructor member;
+}; // expected-error{{a non-type template parameter cannot have type 'class Derived'}}
+   // expected-error at -1{{expected ',' or '>' in template-parameter-list}}
+   // expected-warning at -2{{declaration does not declare anything}}
+