Index: clang/include/clang/Basic/DiagnosticSemaKinds.td =================================================================== --- clang/include/clang/Basic/DiagnosticSemaKinds.td +++ clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -6476,6 +6476,9 @@ def err_bad_memptr_lhs : Error< "left hand operand to %0 must be a %select{|pointer to }1class " "compatible with the right hand operand, but is %2">; +def warn_memptr_incomplete : Warning< + "member pointer has incomplete base type %0">, + InGroup>, DefaultIgnore; def warn_exception_caught_by_earlier_handler : Warning< "exception of type %0 will be caught by earlier handler">, InGroup; Index: clang/lib/Sema/SemaType.cpp =================================================================== --- clang/lib/Sema/SemaType.cpp +++ clang/lib/Sema/SemaType.cpp @@ -2448,6 +2448,12 @@ return QualType(); } + // FIXME: This will cause errors if template instantiation fails. + if (!Context.getDiagnostics().isIgnored(diag::warn_memptr_incomplete, Loc) && + !Class->isDependentType() && + !Class->getAsCXXRecordDecl()->isBeingDefined()) + RequireCompleteType(Loc, Class, diag::warn_memptr_incomplete); + // Adjust the default free function calling convention to the default method // calling convention. bool IsCtorOrDtor = Index: clang/test/SemaCXX/warn-incomplete-member-pointers.cpp =================================================================== --- /dev/null +++ clang/test/SemaCXX/warn-incomplete-member-pointers.cpp @@ -0,0 +1,4 @@ +// RUN: %clang_cc1 -verify -fsyntax-only -Wincomplete-member-pointers %s + +struct S; // expected-note {{forward declaration of 'S'}} +typedef int S::*foo; // expected-warning {{member pointer has incomplete base type 'S'}}