2009-10-11 17:03:14 +08:00
|
|
|
//===--- SemaExceptionSpec.cpp - C++ Exception Specifications ---*- C++ -*-===//
|
|
|
|
//
|
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
//
|
|
|
|
// This file provides Sema routines for C++ exception specification testing.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2010-08-26 06:03:47 +08:00
|
|
|
#include "clang/Sema/SemaInternal.h"
|
2009-10-11 17:03:14 +08:00
|
|
|
#include "clang/AST/CXXInheritance.h"
|
|
|
|
#include "clang/AST/Expr.h"
|
|
|
|
#include "clang/AST/ExprCXX.h"
|
2010-03-24 15:14:45 +08:00
|
|
|
#include "clang/AST/TypeLoc.h"
|
|
|
|
#include "clang/Lex/Preprocessor.h"
|
2010-02-12 15:32:17 +08:00
|
|
|
#include "clang/Basic/Diagnostic.h"
|
|
|
|
#include "clang/Basic/SourceManager.h"
|
2009-10-11 17:03:14 +08:00
|
|
|
#include "llvm/ADT/SmallPtrSet.h"
|
|
|
|
|
|
|
|
namespace clang {
|
|
|
|
|
|
|
|
static const FunctionProtoType *GetUnderlyingFunction(QualType T)
|
|
|
|
{
|
|
|
|
if (const PointerType *PtrTy = T->getAs<PointerType>())
|
|
|
|
T = PtrTy->getPointeeType();
|
|
|
|
else if (const ReferenceType *RefTy = T->getAs<ReferenceType>())
|
|
|
|
T = RefTy->getPointeeType();
|
2009-10-14 22:38:54 +08:00
|
|
|
else if (const MemberPointerType *MPTy = T->getAs<MemberPointerType>())
|
|
|
|
T = MPTy->getPointeeType();
|
2009-10-11 17:03:14 +08:00
|
|
|
return T->getAs<FunctionProtoType>();
|
|
|
|
}
|
|
|
|
|
|
|
|
/// CheckSpecifiedExceptionType - Check if the given type is valid in an
|
|
|
|
/// exception specification. Incomplete types, or pointers to incomplete types
|
|
|
|
/// other than void are not allowed.
|
|
|
|
bool Sema::CheckSpecifiedExceptionType(QualType T, const SourceRange &Range) {
|
|
|
|
|
2009-12-11 02:13:52 +08:00
|
|
|
// This check (and the similar one below) deals with issue 437, that changes
|
|
|
|
// C++ 9.2p2 this way:
|
|
|
|
// Within the class member-specification, the class is regarded as complete
|
|
|
|
// within function bodies, default arguments, exception-specifications, and
|
|
|
|
// constructor ctor-initializers (including such things in nested classes).
|
|
|
|
if (T->isRecordType() && T->getAs<RecordType>()->isBeingDefined())
|
|
|
|
return false;
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
// C++ 15.4p2: A type denoted in an exception-specification shall not denote
|
|
|
|
// an incomplete type.
|
2009-10-14 22:59:48 +08:00
|
|
|
if (RequireCompleteType(Range.getBegin(), T,
|
|
|
|
PDiag(diag::err_incomplete_in_exception_spec) << /*direct*/0 << Range))
|
|
|
|
return true;
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
// C++ 15.4p2: A type denoted in an exception-specification shall not denote
|
|
|
|
// an incomplete type a pointer or reference to an incomplete type, other
|
|
|
|
// than (cv) void*.
|
|
|
|
int kind;
|
|
|
|
if (const PointerType* IT = T->getAs<PointerType>()) {
|
|
|
|
T = IT->getPointeeType();
|
|
|
|
kind = 1;
|
|
|
|
} else if (const ReferenceType* IT = T->getAs<ReferenceType>()) {
|
|
|
|
T = IT->getPointeeType();
|
|
|
|
kind = 2;
|
|
|
|
} else
|
|
|
|
return false;
|
|
|
|
|
2009-12-11 02:13:52 +08:00
|
|
|
// Again as before
|
|
|
|
if (T->isRecordType() && T->getAs<RecordType>()->isBeingDefined())
|
|
|
|
return false;
|
|
|
|
|
2009-10-14 22:59:48 +08:00
|
|
|
if (!T->isVoidType() && RequireCompleteType(Range.getBegin(), T,
|
2009-12-11 02:13:52 +08:00
|
|
|
PDiag(diag::err_incomplete_in_exception_spec) << kind << Range))
|
2009-10-14 22:59:48 +08:00
|
|
|
return true;
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// CheckDistantExceptionSpec - Check if the given type is a pointer or pointer
|
|
|
|
/// to member to a function with an exception specification. This means that
|
|
|
|
/// it is invalid to add another level of indirection.
|
|
|
|
bool Sema::CheckDistantExceptionSpec(QualType T) {
|
|
|
|
if (const PointerType *PT = T->getAs<PointerType>())
|
|
|
|
T = PT->getPointeeType();
|
|
|
|
else if (const MemberPointerType *PT = T->getAs<MemberPointerType>())
|
|
|
|
T = PT->getPointeeType();
|
|
|
|
else
|
|
|
|
return false;
|
|
|
|
|
|
|
|
const FunctionProtoType *FnT = T->getAs<FunctionProtoType>();
|
|
|
|
if (!FnT)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
return FnT->hasExceptionSpec();
|
|
|
|
}
|
|
|
|
|
2010-02-12 15:32:17 +08:00
|
|
|
bool Sema::CheckEquivalentExceptionSpec(FunctionDecl *Old, FunctionDecl *New) {
|
2011-03-16 03:52:30 +08:00
|
|
|
OverloadedOperatorKind OO = New->getDeclName().getCXXOverloadedOperator();
|
|
|
|
bool IsOperatorNew = OO == OO_New || OO == OO_Array_New;
|
2010-03-24 15:14:45 +08:00
|
|
|
bool MissingExceptionSpecification = false;
|
2010-02-12 15:32:17 +08:00
|
|
|
bool MissingEmptyExceptionSpecification = false;
|
2011-03-20 07:05:18 +08:00
|
|
|
unsigned DiagID = diag::err_mismatched_exception_spec;
|
2011-09-18 01:15:52 +08:00
|
|
|
if (getLangOptions().MicrosoftExt)
|
2011-04-22 16:25:24 +08:00
|
|
|
DiagID = diag::warn_mismatched_exception_spec;
|
2011-03-20 07:05:18 +08:00
|
|
|
|
|
|
|
if (!CheckEquivalentExceptionSpec(PDiag(DiagID),
|
2010-03-30 07:34:08 +08:00
|
|
|
PDiag(diag::note_previous_declaration),
|
2010-02-12 15:32:17 +08:00
|
|
|
Old->getType()->getAs<FunctionProtoType>(),
|
|
|
|
Old->getLocation(),
|
|
|
|
New->getType()->getAs<FunctionProtoType>(),
|
|
|
|
New->getLocation(),
|
2010-03-24 15:14:45 +08:00
|
|
|
&MissingExceptionSpecification,
|
2011-03-12 19:50:43 +08:00
|
|
|
&MissingEmptyExceptionSpecification,
|
2011-03-16 03:52:30 +08:00
|
|
|
/*AllowNoexceptAllMatchWithNoSpec=*/true,
|
|
|
|
IsOperatorNew))
|
2010-02-12 15:32:17 +08:00
|
|
|
return false;
|
|
|
|
|
|
|
|
// The failure was something other than an empty exception
|
|
|
|
// specification; return an error.
|
2010-03-24 15:14:45 +08:00
|
|
|
if (!MissingExceptionSpecification && !MissingEmptyExceptionSpecification)
|
2010-02-12 15:32:17 +08:00
|
|
|
return true;
|
|
|
|
|
2010-12-14 16:05:40 +08:00
|
|
|
const FunctionProtoType *NewProto
|
|
|
|
= New->getType()->getAs<FunctionProtoType>();
|
|
|
|
|
2010-02-12 15:32:17 +08:00
|
|
|
// The new function declaration is only missing an empty exception
|
|
|
|
// specification "throw()". If the throw() specification came from a
|
|
|
|
// function in a system header that has C linkage, just add an empty
|
|
|
|
// exception specification to the "new" declaration. This is an
|
|
|
|
// egregious workaround for glibc, which adds throw() specifications
|
|
|
|
// to many libc functions as an optimization. Unfortunately, that
|
|
|
|
// optimization isn't permitted by the C++ standard, so we're forced
|
|
|
|
// to work around it here.
|
2010-12-14 16:05:40 +08:00
|
|
|
if (MissingEmptyExceptionSpecification && NewProto &&
|
2010-03-24 15:14:45 +08:00
|
|
|
(Old->getLocation().isInvalid() ||
|
|
|
|
Context.getSourceManager().isInSystemHeader(Old->getLocation())) &&
|
2010-02-12 15:32:17 +08:00
|
|
|
Old->isExternC()) {
|
2010-12-14 16:05:40 +08:00
|
|
|
FunctionProtoType::ExtProtoInfo EPI = NewProto->getExtProtoInfo();
|
2011-03-12 19:50:43 +08:00
|
|
|
EPI.ExceptionSpecType = EST_DynamicNone;
|
2010-02-12 15:32:17 +08:00
|
|
|
QualType NewType = Context.getFunctionType(NewProto->getResultType(),
|
|
|
|
NewProto->arg_type_begin(),
|
|
|
|
NewProto->getNumArgs(),
|
2010-12-14 16:05:40 +08:00
|
|
|
EPI);
|
2010-02-12 15:32:17 +08:00
|
|
|
New->setType(NewType);
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2010-12-14 16:05:40 +08:00
|
|
|
if (MissingExceptionSpecification && NewProto) {
|
2010-03-24 15:14:45 +08:00
|
|
|
const FunctionProtoType *OldProto
|
|
|
|
= Old->getType()->getAs<FunctionProtoType>();
|
|
|
|
|
2010-12-14 16:05:40 +08:00
|
|
|
FunctionProtoType::ExtProtoInfo EPI = NewProto->getExtProtoInfo();
|
2011-03-12 19:50:43 +08:00
|
|
|
EPI.ExceptionSpecType = OldProto->getExceptionSpecType();
|
|
|
|
if (EPI.ExceptionSpecType == EST_Dynamic) {
|
|
|
|
EPI.NumExceptions = OldProto->getNumExceptions();
|
|
|
|
EPI.Exceptions = OldProto->exception_begin();
|
|
|
|
} else if (EPI.ExceptionSpecType == EST_ComputedNoexcept) {
|
|
|
|
// FIXME: We can't just take the expression from the old prototype. It
|
|
|
|
// likely contains references to the old prototype's parameters.
|
|
|
|
}
|
2010-12-14 16:05:40 +08:00
|
|
|
|
2010-03-24 15:14:45 +08:00
|
|
|
// Update the type of the function with the appropriate exception
|
|
|
|
// specification.
|
|
|
|
QualType NewType = Context.getFunctionType(NewProto->getResultType(),
|
|
|
|
NewProto->arg_type_begin(),
|
|
|
|
NewProto->getNumArgs(),
|
2010-12-14 16:05:40 +08:00
|
|
|
EPI);
|
2010-03-24 15:14:45 +08:00
|
|
|
New->setType(NewType);
|
|
|
|
|
|
|
|
// If exceptions are disabled, suppress the warning about missing
|
|
|
|
// exception specifications for new and delete operators.
|
2011-03-02 10:04:40 +08:00
|
|
|
if (!getLangOptions().CXXExceptions) {
|
2010-03-24 15:14:45 +08:00
|
|
|
switch (New->getDeclName().getCXXOverloadedOperator()) {
|
|
|
|
case OO_New:
|
|
|
|
case OO_Array_New:
|
|
|
|
case OO_Delete:
|
|
|
|
case OO_Array_Delete:
|
|
|
|
if (New->getDeclContext()->isTranslationUnit())
|
|
|
|
return false;
|
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Warn about the lack of exception specification.
|
|
|
|
llvm::SmallString<128> ExceptionSpecString;
|
|
|
|
llvm::raw_svector_ostream OS(ExceptionSpecString);
|
2011-03-12 19:50:43 +08:00
|
|
|
switch (OldProto->getExceptionSpecType()) {
|
|
|
|
case EST_DynamicNone:
|
|
|
|
OS << "throw()";
|
|
|
|
break;
|
|
|
|
|
|
|
|
case EST_Dynamic: {
|
|
|
|
OS << "throw(";
|
|
|
|
bool OnFirstException = true;
|
|
|
|
for (FunctionProtoType::exception_iterator E = OldProto->exception_begin(),
|
|
|
|
EEnd = OldProto->exception_end();
|
|
|
|
E != EEnd;
|
|
|
|
++E) {
|
|
|
|
if (OnFirstException)
|
|
|
|
OnFirstException = false;
|
|
|
|
else
|
|
|
|
OS << ", ";
|
|
|
|
|
2011-09-28 07:30:47 +08:00
|
|
|
OS << E->getAsString(getPrintingPolicy());
|
2011-03-12 19:50:43 +08:00
|
|
|
}
|
|
|
|
OS << ")";
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
case EST_BasicNoexcept:
|
|
|
|
OS << "noexcept";
|
|
|
|
break;
|
|
|
|
|
|
|
|
case EST_ComputedNoexcept:
|
|
|
|
OS << "noexcept(";
|
2011-09-28 07:30:47 +08:00
|
|
|
OldProto->getNoexceptExpr()->printPretty(OS, Context, 0,
|
|
|
|
getPrintingPolicy());
|
2011-03-12 19:50:43 +08:00
|
|
|
OS << ")";
|
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
2011-09-23 13:06:16 +08:00
|
|
|
llvm_unreachable("This spec type is compatible with none.");
|
2010-03-24 15:14:45 +08:00
|
|
|
}
|
|
|
|
OS.flush();
|
|
|
|
|
2011-03-12 19:17:06 +08:00
|
|
|
SourceLocation FixItLoc;
|
2010-03-24 15:14:45 +08:00
|
|
|
if (TypeSourceInfo *TSInfo = New->getTypeSourceInfo()) {
|
2010-12-15 06:11:44 +08:00
|
|
|
TypeLoc TL = TSInfo->getTypeLoc().IgnoreParens();
|
2010-03-24 15:14:45 +08:00
|
|
|
if (const FunctionTypeLoc *FTLoc = dyn_cast<FunctionTypeLoc>(&TL))
|
2011-03-12 19:17:06 +08:00
|
|
|
FixItLoc = PP.getLocForEndOfToken(FTLoc->getLocalRangeEnd());
|
2010-03-24 15:14:45 +08:00
|
|
|
}
|
|
|
|
|
2011-03-12 19:17:06 +08:00
|
|
|
if (FixItLoc.isInvalid())
|
2010-03-24 15:14:45 +08:00
|
|
|
Diag(New->getLocation(), diag::warn_missing_exception_specification)
|
|
|
|
<< New << OS.str();
|
|
|
|
else {
|
|
|
|
// FIXME: This will get more complicated with C++0x
|
|
|
|
// late-specified return types.
|
|
|
|
Diag(New->getLocation(), diag::warn_missing_exception_specification)
|
|
|
|
<< New << OS.str()
|
2011-03-12 19:17:06 +08:00
|
|
|
<< FixItHint::CreateInsertion(FixItLoc, " " + OS.str().str());
|
2010-03-24 15:14:45 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
if (!Old->getLocation().isInvalid())
|
|
|
|
Diag(Old->getLocation(), diag::note_previous_declaration);
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2011-03-20 07:05:18 +08:00
|
|
|
Diag(New->getLocation(), DiagID);
|
2010-02-12 15:32:17 +08:00
|
|
|
Diag(Old->getLocation(), diag::note_previous_declaration);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
/// CheckEquivalentExceptionSpec - Check if the two types have equivalent
|
|
|
|
/// exception specifications. Exception specifications are equivalent if
|
|
|
|
/// they allow exactly the same set of exception types. It does not matter how
|
|
|
|
/// that is achieved. See C++ [except.spec]p2.
|
|
|
|
bool Sema::CheckEquivalentExceptionSpec(
|
|
|
|
const FunctionProtoType *Old, SourceLocation OldLoc,
|
|
|
|
const FunctionProtoType *New, SourceLocation NewLoc) {
|
2011-03-20 07:05:18 +08:00
|
|
|
unsigned DiagID = diag::err_mismatched_exception_spec;
|
2011-09-18 01:15:52 +08:00
|
|
|
if (getLangOptions().MicrosoftExt)
|
2011-04-22 16:25:24 +08:00
|
|
|
DiagID = diag::warn_mismatched_exception_spec;
|
2010-03-30 07:34:08 +08:00
|
|
|
return CheckEquivalentExceptionSpec(
|
2011-03-20 07:05:18 +08:00
|
|
|
PDiag(DiagID),
|
2010-03-30 07:34:08 +08:00
|
|
|
PDiag(diag::note_previous_declaration),
|
2009-10-11 17:03:14 +08:00
|
|
|
Old, OldLoc, New, NewLoc);
|
|
|
|
}
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
/// CheckEquivalentExceptionSpec - Check if the two types have compatible
|
|
|
|
/// exception specifications. See C++ [except.spec]p3.
|
|
|
|
bool Sema::CheckEquivalentExceptionSpec(const PartialDiagnostic &DiagID,
|
2010-03-24 15:14:45 +08:00
|
|
|
const PartialDiagnostic & NoteID,
|
2011-03-12 19:50:43 +08:00
|
|
|
const FunctionProtoType *Old,
|
2010-03-24 15:14:45 +08:00
|
|
|
SourceLocation OldLoc,
|
2011-03-12 19:50:43 +08:00
|
|
|
const FunctionProtoType *New,
|
2010-03-24 15:14:45 +08:00
|
|
|
SourceLocation NewLoc,
|
|
|
|
bool *MissingExceptionSpecification,
|
2011-03-12 19:50:43 +08:00
|
|
|
bool*MissingEmptyExceptionSpecification,
|
2011-03-16 03:52:30 +08:00
|
|
|
bool AllowNoexceptAllMatchWithNoSpec,
|
|
|
|
bool IsOperatorNew) {
|
2010-05-28 16:37:35 +08:00
|
|
|
// Just completely ignore this under -fno-exceptions.
|
2011-03-02 10:04:40 +08:00
|
|
|
if (!getLangOptions().CXXExceptions)
|
2010-05-28 16:37:35 +08:00
|
|
|
return false;
|
|
|
|
|
2010-03-24 15:14:45 +08:00
|
|
|
if (MissingExceptionSpecification)
|
|
|
|
*MissingExceptionSpecification = false;
|
|
|
|
|
2010-02-12 15:32:17 +08:00
|
|
|
if (MissingEmptyExceptionSpecification)
|
|
|
|
*MissingEmptyExceptionSpecification = false;
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
// C++0x [except.spec]p3: Two exception-specifications are compatible if:
|
|
|
|
// - both are non-throwing, regardless of their form,
|
|
|
|
// - both have the form noexcept(constant-expression) and the constant-
|
|
|
|
// expressions are equivalent,
|
|
|
|
// - both are dynamic-exception-specifications that have the same set of
|
|
|
|
// adjusted types.
|
|
|
|
//
|
|
|
|
// C++0x [except.spec]p12: An exception-specifcation is non-throwing if it is
|
|
|
|
// of the form throw(), noexcept, or noexcept(constant-expression) where the
|
|
|
|
// constant-expression yields true.
|
|
|
|
//
|
|
|
|
// C++0x [except.spec]p4: If any declaration of a function has an exception-
|
|
|
|
// specifier that is not a noexcept-specification allowing all exceptions,
|
|
|
|
// all declarations [...] of that function shall have a compatible
|
|
|
|
// exception-specification.
|
|
|
|
//
|
|
|
|
// That last point basically means that noexcept(false) matches no spec.
|
|
|
|
// It's considered when AllowNoexceptAllMatchWithNoSpec is true.
|
|
|
|
|
|
|
|
ExceptionSpecificationType OldEST = Old->getExceptionSpecType();
|
|
|
|
ExceptionSpecificationType NewEST = New->getExceptionSpecType();
|
|
|
|
|
2011-06-12 01:19:42 +08:00
|
|
|
assert(OldEST != EST_Delayed && NewEST != EST_Delayed &&
|
|
|
|
"Shouldn't see unknown exception specifications here");
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
// Shortcut the case where both have no spec.
|
|
|
|
if (OldEST == EST_None && NewEST == EST_None)
|
|
|
|
return false;
|
|
|
|
|
2011-03-14 01:09:40 +08:00
|
|
|
FunctionProtoType::NoexceptResult OldNR = Old->getNoexceptSpec(Context);
|
|
|
|
FunctionProtoType::NoexceptResult NewNR = New->getNoexceptSpec(Context);
|
2011-03-12 19:50:43 +08:00
|
|
|
if (OldNR == FunctionProtoType::NR_BadNoexcept ||
|
|
|
|
NewNR == FunctionProtoType::NR_BadNoexcept)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Dependent noexcept specifiers are compatible with each other, but nothing
|
|
|
|
// else.
|
|
|
|
// One noexcept is compatible with another if the argument is the same
|
|
|
|
if (OldNR == NewNR &&
|
|
|
|
OldNR != FunctionProtoType::NR_NoNoexcept &&
|
|
|
|
NewNR != FunctionProtoType::NR_NoNoexcept)
|
|
|
|
return false;
|
|
|
|
if (OldNR != NewNR &&
|
|
|
|
OldNR != FunctionProtoType::NR_NoNoexcept &&
|
|
|
|
NewNR != FunctionProtoType::NR_NoNoexcept) {
|
|
|
|
Diag(NewLoc, DiagID);
|
|
|
|
if (NoteID.getDiagID() != 0)
|
|
|
|
Diag(OldLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// The MS extension throw(...) is compatible with itself.
|
|
|
|
if (OldEST == EST_MSAny && NewEST == EST_MSAny)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// It's also compatible with no spec.
|
|
|
|
if ((OldEST == EST_None && NewEST == EST_MSAny) ||
|
|
|
|
(OldEST == EST_MSAny && NewEST == EST_None))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// It's also compatible with noexcept(false).
|
|
|
|
if (OldEST == EST_MSAny && NewNR == FunctionProtoType::NR_Throw)
|
|
|
|
return false;
|
|
|
|
if (NewEST == EST_MSAny && OldNR == FunctionProtoType::NR_Throw)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// As described above, noexcept(false) matches no spec only for functions.
|
|
|
|
if (AllowNoexceptAllMatchWithNoSpec) {
|
|
|
|
if (OldEST == EST_None && NewNR == FunctionProtoType::NR_Throw)
|
|
|
|
return false;
|
|
|
|
if (NewEST == EST_None && OldNR == FunctionProtoType::NR_Throw)
|
|
|
|
return false;
|
2010-08-30 23:04:51 +08:00
|
|
|
}
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
// Any non-throwing specifications are compatible.
|
|
|
|
bool OldNonThrowing = OldNR == FunctionProtoType::NR_Nothrow ||
|
|
|
|
OldEST == EST_DynamicNone;
|
|
|
|
bool NewNonThrowing = NewNR == FunctionProtoType::NR_Nothrow ||
|
|
|
|
NewEST == EST_DynamicNone;
|
|
|
|
if (OldNonThrowing && NewNonThrowing)
|
2009-10-11 17:03:14 +08:00
|
|
|
return false;
|
2011-03-12 19:50:43 +08:00
|
|
|
|
2011-03-16 03:52:30 +08:00
|
|
|
// As a special compatibility feature, under C++0x we accept no spec and
|
|
|
|
// throw(std::bad_alloc) as equivalent for operator new and operator new[].
|
|
|
|
// This is because the implicit declaration changed, but old code would break.
|
|
|
|
if (getLangOptions().CPlusPlus0x && IsOperatorNew) {
|
|
|
|
const FunctionProtoType *WithExceptions = 0;
|
|
|
|
if (OldEST == EST_None && NewEST == EST_Dynamic)
|
|
|
|
WithExceptions = New;
|
|
|
|
else if (OldEST == EST_Dynamic && NewEST == EST_None)
|
|
|
|
WithExceptions = Old;
|
|
|
|
if (WithExceptions && WithExceptions->getNumExceptions() == 1) {
|
|
|
|
// One has no spec, the other throw(something). If that something is
|
|
|
|
// std::bad_alloc, all conditions are met.
|
|
|
|
QualType Exception = *WithExceptions->exception_begin();
|
|
|
|
if (CXXRecordDecl *ExRecord = Exception->getAsCXXRecordDecl()) {
|
|
|
|
IdentifierInfo* Name = ExRecord->getIdentifier();
|
|
|
|
if (Name && Name->getName() == "bad_alloc") {
|
|
|
|
// It's called bad_alloc, but is it in std?
|
|
|
|
DeclContext* DC = ExRecord->getDeclContext();
|
2011-03-16 04:41:09 +08:00
|
|
|
DC = DC->getEnclosingNamespaceContext();
|
|
|
|
if (NamespaceDecl* NS = dyn_cast<NamespaceDecl>(DC)) {
|
2011-03-16 03:52:30 +08:00
|
|
|
IdentifierInfo* NSName = NS->getIdentifier();
|
2011-03-16 04:41:09 +08:00
|
|
|
DC = DC->getParent();
|
2011-03-16 03:52:30 +08:00
|
|
|
if (NSName && NSName->getName() == "std" &&
|
2011-03-16 04:41:09 +08:00
|
|
|
DC->getEnclosingNamespaceContext()->isTranslationUnit()) {
|
2011-03-16 03:52:30 +08:00
|
|
|
return false;
|
2011-03-16 04:41:09 +08:00
|
|
|
}
|
2011-03-16 03:52:30 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
// At this point, the only remaining valid case is two matching dynamic
|
|
|
|
// specifications. We return here unless both specifications are dynamic.
|
|
|
|
if (OldEST != EST_Dynamic || NewEST != EST_Dynamic) {
|
2010-03-24 15:14:45 +08:00
|
|
|
if (MissingExceptionSpecification && Old->hasExceptionSpec() &&
|
2010-02-12 15:32:17 +08:00
|
|
|
!New->hasExceptionSpec()) {
|
2010-03-24 15:14:45 +08:00
|
|
|
// The old type has an exception specification of some sort, but
|
|
|
|
// the new type does not.
|
|
|
|
*MissingExceptionSpecification = true;
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
if (MissingEmptyExceptionSpecification && OldNonThrowing) {
|
|
|
|
// The old type has a throw() or noexcept(true) exception specification
|
|
|
|
// and the new type has no exception specification, and the caller asked
|
2010-03-24 15:14:45 +08:00
|
|
|
// to handle this itself.
|
|
|
|
*MissingEmptyExceptionSpecification = true;
|
|
|
|
}
|
|
|
|
|
2010-02-12 15:32:17 +08:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(NewLoc, DiagID);
|
2009-10-15 00:09:29 +08:00
|
|
|
if (NoteID.getDiagID() != 0)
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(OldLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
assert(OldEST == EST_Dynamic && NewEST == EST_Dynamic &&
|
|
|
|
"Exception compatibility logic error: non-dynamic spec slipped through.");
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
bool Success = true;
|
2011-03-12 19:50:43 +08:00
|
|
|
// Both have a dynamic exception spec. Collect the first set, then compare
|
2009-10-11 17:03:14 +08:00
|
|
|
// to the second.
|
2009-10-14 23:06:25 +08:00
|
|
|
llvm::SmallPtrSet<CanQualType, 8> OldTypes, NewTypes;
|
2009-10-11 17:03:14 +08:00
|
|
|
for (FunctionProtoType::exception_iterator I = Old->exception_begin(),
|
|
|
|
E = Old->exception_end(); I != E; ++I)
|
2009-10-14 23:06:25 +08:00
|
|
|
OldTypes.insert(Context.getCanonicalType(*I).getUnqualifiedType());
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
for (FunctionProtoType::exception_iterator I = New->exception_begin(),
|
2009-10-11 17:11:23 +08:00
|
|
|
E = New->exception_end(); I != E && Success; ++I) {
|
2009-10-14 23:06:25 +08:00
|
|
|
CanQualType TypePtr = Context.getCanonicalType(*I).getUnqualifiedType();
|
2009-10-11 17:11:23 +08:00
|
|
|
if(OldTypes.count(TypePtr))
|
|
|
|
NewTypes.insert(TypePtr);
|
|
|
|
else
|
|
|
|
Success = false;
|
|
|
|
}
|
2009-10-11 17:03:14 +08:00
|
|
|
|
2009-10-11 17:11:23 +08:00
|
|
|
Success = Success && OldTypes.size() == NewTypes.size();
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
if (Success) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
Diag(NewLoc, DiagID);
|
2009-10-15 00:09:29 +08:00
|
|
|
if (NoteID.getDiagID() != 0)
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(OldLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// CheckExceptionSpecSubset - Check whether the second function type's
|
|
|
|
/// exception specification is a subset (or equivalent) of the first function
|
|
|
|
/// type. This is used by override and pointer assignment checks.
|
2009-10-15 00:09:29 +08:00
|
|
|
bool Sema::CheckExceptionSpecSubset(
|
|
|
|
const PartialDiagnostic &DiagID, const PartialDiagnostic & NoteID,
|
2009-10-11 17:03:14 +08:00
|
|
|
const FunctionProtoType *Superset, SourceLocation SuperLoc,
|
|
|
|
const FunctionProtoType *Subset, SourceLocation SubLoc) {
|
2010-05-28 16:37:35 +08:00
|
|
|
|
|
|
|
// Just auto-succeed under -fno-exceptions.
|
2011-03-02 10:04:40 +08:00
|
|
|
if (!getLangOptions().CXXExceptions)
|
2010-05-28 16:37:35 +08:00
|
|
|
return false;
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
// FIXME: As usual, we could be more specific in our error messages, but
|
|
|
|
// that better waits until we've got types with source locations.
|
|
|
|
|
|
|
|
if (!SubLoc.isValid())
|
|
|
|
SubLoc = SuperLoc;
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
ExceptionSpecificationType SuperEST = Superset->getExceptionSpecType();
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
// If superset contains everything, we're done.
|
2011-03-12 19:50:43 +08:00
|
|
|
if (SuperEST == EST_None || SuperEST == EST_MSAny)
|
|
|
|
return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
|
|
|
|
|
|
|
|
// If there are dependent noexcept specs, assume everything is fine. Unlike
|
|
|
|
// with the equivalency check, this is safe in this case, because we don't
|
|
|
|
// want to merge declarations. Checks after instantiation will catch any
|
|
|
|
// omissions we make here.
|
|
|
|
// We also shortcut checking if a noexcept expression was bad.
|
|
|
|
|
2011-03-14 01:09:40 +08:00
|
|
|
FunctionProtoType::NoexceptResult SuperNR =Superset->getNoexceptSpec(Context);
|
2011-03-12 19:50:43 +08:00
|
|
|
if (SuperNR == FunctionProtoType::NR_BadNoexcept ||
|
|
|
|
SuperNR == FunctionProtoType::NR_Dependent)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Another case of the superset containing everything.
|
|
|
|
if (SuperNR == FunctionProtoType::NR_Throw)
|
2009-10-11 17:03:14 +08:00
|
|
|
return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
ExceptionSpecificationType SubEST = Subset->getExceptionSpecType();
|
|
|
|
|
2011-06-12 01:19:42 +08:00
|
|
|
assert(SuperEST != EST_Delayed && SubEST != EST_Delayed &&
|
|
|
|
"Shouldn't see unknown exception specifications here");
|
|
|
|
|
2009-10-11 17:03:14 +08:00
|
|
|
// It does not. If the subset contains everything, we've failed.
|
2011-03-12 19:50:43 +08:00
|
|
|
if (SubEST == EST_None || SubEST == EST_MSAny) {
|
|
|
|
Diag(SubLoc, DiagID);
|
|
|
|
if (NoteID.getDiagID() != 0)
|
|
|
|
Diag(SuperLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2011-03-14 01:09:40 +08:00
|
|
|
FunctionProtoType::NoexceptResult SubNR = Subset->getNoexceptSpec(Context);
|
2011-03-12 19:50:43 +08:00
|
|
|
if (SubNR == FunctionProtoType::NR_BadNoexcept ||
|
|
|
|
SubNR == FunctionProtoType::NR_Dependent)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Another case of the subset containing everything.
|
|
|
|
if (SubNR == FunctionProtoType::NR_Throw) {
|
|
|
|
Diag(SubLoc, DiagID);
|
|
|
|
if (NoteID.getDiagID() != 0)
|
|
|
|
Diag(SuperLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If the subset contains nothing, we're done.
|
|
|
|
if (SubEST == EST_DynamicNone || SubNR == FunctionProtoType::NR_Nothrow)
|
|
|
|
return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
|
|
|
|
|
|
|
|
// Otherwise, if the superset contains nothing, we've failed.
|
|
|
|
if (SuperEST == EST_DynamicNone || SuperNR == FunctionProtoType::NR_Nothrow) {
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(SubLoc, DiagID);
|
2009-10-15 00:09:29 +08:00
|
|
|
if (NoteID.getDiagID() != 0)
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(SuperLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2011-03-12 19:50:43 +08:00
|
|
|
assert(SuperEST == EST_Dynamic && SubEST == EST_Dynamic &&
|
|
|
|
"Exception spec subset: non-dynamic case slipped through.");
|
|
|
|
|
|
|
|
// Neither contains everything or nothing. Do a proper comparison.
|
2009-10-11 17:03:14 +08:00
|
|
|
for (FunctionProtoType::exception_iterator SubI = Subset->exception_begin(),
|
|
|
|
SubE = Subset->exception_end(); SubI != SubE; ++SubI) {
|
|
|
|
// Take one type from the subset.
|
|
|
|
QualType CanonicalSubT = Context.getCanonicalType(*SubI);
|
2009-10-14 22:38:54 +08:00
|
|
|
// Unwrap pointers and references so that we can do checks within a class
|
|
|
|
// hierarchy. Don't unwrap member pointers; they don't have hierarchy
|
|
|
|
// conversions on the pointee.
|
2009-10-11 17:03:14 +08:00
|
|
|
bool SubIsPointer = false;
|
|
|
|
if (const ReferenceType *RefTy = CanonicalSubT->getAs<ReferenceType>())
|
|
|
|
CanonicalSubT = RefTy->getPointeeType();
|
|
|
|
if (const PointerType *PtrTy = CanonicalSubT->getAs<PointerType>()) {
|
|
|
|
CanonicalSubT = PtrTy->getPointeeType();
|
|
|
|
SubIsPointer = true;
|
|
|
|
}
|
|
|
|
bool SubIsClass = CanonicalSubT->isRecordType();
|
First part of changes to eliminate problems with cv-qualifiers and
sugared types. The basic problem is that our qualifier accessors
(getQualifiers, getCVRQualifiers, isConstQualified, etc.) only look at
the current QualType and not at any qualifiers that come from sugared
types, meaning that we won't see these qualifiers through, e.g.,
typedefs:
typedef const int CInt;
typedef CInt Self;
Self.isConstQualified() currently returns false!
Various bugs (e.g., PR5383) have cropped up all over the front end due
to such problems. I'm addressing this problem by splitting each
qualifier accessor into two versions:
- the "local" version only returns qualifiers on this particular
QualType instance
- the "normal" version that will eventually combine qualifiers from this
QualType instance with the qualifiers on the canonical type to
produce the full set of qualifiers.
This commit adds the local versions and switches a few callers from
the "normal" version (e.g., isConstQualified) over to the "local"
version (e.g., isLocalConstQualified) when that is the right thing to
do, e.g., because we're printing or serializing the qualifiers. Also,
switch a bunch of
Context.getCanonicalType(T1).getUnqualifiedType() == Context.getCanonicalType(T2).getQualifiedType()
expressions over to
Context.hasSameUnqualifiedType(T1, T2)
llvm-svn: 88969
2009-11-17 05:35:15 +08:00
|
|
|
CanonicalSubT = CanonicalSubT.getLocalUnqualifiedType();
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
CXXBasePaths Paths(/*FindAmbiguities=*/true, /*RecordPaths=*/true,
|
|
|
|
/*DetectVirtual=*/false);
|
|
|
|
|
|
|
|
bool Contained = false;
|
|
|
|
// Make sure it's in the superset.
|
|
|
|
for (FunctionProtoType::exception_iterator SuperI =
|
|
|
|
Superset->exception_begin(), SuperE = Superset->exception_end();
|
|
|
|
SuperI != SuperE; ++SuperI) {
|
|
|
|
QualType CanonicalSuperT = Context.getCanonicalType(*SuperI);
|
|
|
|
// SubT must be SuperT or derived from it, or pointer or reference to
|
|
|
|
// such types.
|
|
|
|
if (const ReferenceType *RefTy = CanonicalSuperT->getAs<ReferenceType>())
|
|
|
|
CanonicalSuperT = RefTy->getPointeeType();
|
|
|
|
if (SubIsPointer) {
|
|
|
|
if (const PointerType *PtrTy = CanonicalSuperT->getAs<PointerType>())
|
|
|
|
CanonicalSuperT = PtrTy->getPointeeType();
|
|
|
|
else {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
First part of changes to eliminate problems with cv-qualifiers and
sugared types. The basic problem is that our qualifier accessors
(getQualifiers, getCVRQualifiers, isConstQualified, etc.) only look at
the current QualType and not at any qualifiers that come from sugared
types, meaning that we won't see these qualifiers through, e.g.,
typedefs:
typedef const int CInt;
typedef CInt Self;
Self.isConstQualified() currently returns false!
Various bugs (e.g., PR5383) have cropped up all over the front end due
to such problems. I'm addressing this problem by splitting each
qualifier accessor into two versions:
- the "local" version only returns qualifiers on this particular
QualType instance
- the "normal" version that will eventually combine qualifiers from this
QualType instance with the qualifiers on the canonical type to
produce the full set of qualifiers.
This commit adds the local versions and switches a few callers from
the "normal" version (e.g., isConstQualified) over to the "local"
version (e.g., isLocalConstQualified) when that is the right thing to
do, e.g., because we're printing or serializing the qualifiers. Also,
switch a bunch of
Context.getCanonicalType(T1).getUnqualifiedType() == Context.getCanonicalType(T2).getQualifiedType()
expressions over to
Context.hasSameUnqualifiedType(T1, T2)
llvm-svn: 88969
2009-11-17 05:35:15 +08:00
|
|
|
CanonicalSuperT = CanonicalSuperT.getLocalUnqualifiedType();
|
2009-10-11 17:03:14 +08:00
|
|
|
// If the types are the same, move on to the next type in the subset.
|
|
|
|
if (CanonicalSubT == CanonicalSuperT) {
|
|
|
|
Contained = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Otherwise we need to check the inheritance.
|
|
|
|
if (!SubIsClass || !CanonicalSuperT->isRecordType())
|
|
|
|
continue;
|
|
|
|
|
|
|
|
Paths.clear();
|
|
|
|
if (!IsDerivedFrom(CanonicalSubT, CanonicalSuperT, Paths))
|
|
|
|
continue;
|
|
|
|
|
2010-05-22 04:29:55 +08:00
|
|
|
if (Paths.isAmbiguous(Context.getCanonicalType(CanonicalSuperT)))
|
2009-10-11 17:03:14 +08:00
|
|
|
continue;
|
|
|
|
|
2010-02-10 17:31:12 +08:00
|
|
|
// Do this check from a context without privileges.
|
2010-03-16 13:22:47 +08:00
|
|
|
switch (CheckBaseClassAccess(SourceLocation(),
|
2010-02-10 17:31:12 +08:00
|
|
|
CanonicalSuperT, CanonicalSubT,
|
|
|
|
Paths.front(),
|
2010-03-16 13:22:47 +08:00
|
|
|
/*Diagnostic*/ 0,
|
2010-02-10 17:31:12 +08:00
|
|
|
/*ForceCheck*/ true,
|
2010-03-16 13:22:47 +08:00
|
|
|
/*ForceUnprivileged*/ true)) {
|
2010-02-10 17:31:12 +08:00
|
|
|
case AR_accessible: break;
|
|
|
|
case AR_inaccessible: continue;
|
|
|
|
case AR_dependent:
|
|
|
|
llvm_unreachable("access check dependent for unprivileged context");
|
|
|
|
break;
|
|
|
|
case AR_delayed:
|
|
|
|
llvm_unreachable("access check delayed in non-declaration");
|
|
|
|
break;
|
|
|
|
}
|
2009-10-11 17:03:14 +08:00
|
|
|
|
|
|
|
Contained = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
if (!Contained) {
|
|
|
|
Diag(SubLoc, DiagID);
|
2009-10-15 00:09:29 +08:00
|
|
|
if (NoteID.getDiagID() != 0)
|
2009-10-11 17:03:14 +08:00
|
|
|
Diag(SuperLoc, NoteID);
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// We've run half the gauntlet.
|
|
|
|
return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool CheckSpecForTypesEquivalent(Sema &S,
|
2009-10-15 00:09:29 +08:00
|
|
|
const PartialDiagnostic &DiagID, const PartialDiagnostic & NoteID,
|
2009-10-11 17:03:14 +08:00
|
|
|
QualType Target, SourceLocation TargetLoc,
|
|
|
|
QualType Source, SourceLocation SourceLoc)
|
|
|
|
{
|
|
|
|
const FunctionProtoType *TFunc = GetUnderlyingFunction(Target);
|
|
|
|
if (!TFunc)
|
|
|
|
return false;
|
|
|
|
const FunctionProtoType *SFunc = GetUnderlyingFunction(Source);
|
|
|
|
if (!SFunc)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
return S.CheckEquivalentExceptionSpec(DiagID, NoteID, TFunc, TargetLoc,
|
|
|
|
SFunc, SourceLoc);
|
|
|
|
}
|
|
|
|
|
|
|
|
/// CheckParamExceptionSpec - Check if the parameter and return types of the
|
|
|
|
/// two functions have equivalent exception specs. This is part of the
|
|
|
|
/// assignment and override compatibility check. We do not check the parameters
|
|
|
|
/// of parameter function pointers recursively, as no sane programmer would
|
|
|
|
/// even be able to write such a function type.
|
2009-10-15 00:09:29 +08:00
|
|
|
bool Sema::CheckParamExceptionSpec(const PartialDiagnostic & NoteID,
|
2009-10-11 17:03:14 +08:00
|
|
|
const FunctionProtoType *Target, SourceLocation TargetLoc,
|
|
|
|
const FunctionProtoType *Source, SourceLocation SourceLoc)
|
|
|
|
{
|
2009-10-15 00:09:29 +08:00
|
|
|
if (CheckSpecForTypesEquivalent(*this,
|
2010-03-30 07:34:08 +08:00
|
|
|
PDiag(diag::err_deep_exception_specs_differ) << 0,
|
|
|
|
PDiag(),
|
2009-10-11 17:03:14 +08:00
|
|
|
Target->getResultType(), TargetLoc,
|
|
|
|
Source->getResultType(), SourceLoc))
|
|
|
|
return true;
|
|
|
|
|
2009-10-15 00:09:29 +08:00
|
|
|
// We shouldn't even be testing this unless the arguments are otherwise
|
2009-10-11 17:03:14 +08:00
|
|
|
// compatible.
|
|
|
|
assert(Target->getNumArgs() == Source->getNumArgs() &&
|
|
|
|
"Functions have different argument counts.");
|
|
|
|
for (unsigned i = 0, E = Target->getNumArgs(); i != E; ++i) {
|
2009-10-15 00:09:29 +08:00
|
|
|
if (CheckSpecForTypesEquivalent(*this,
|
2010-03-30 07:34:08 +08:00
|
|
|
PDiag(diag::err_deep_exception_specs_differ) << 1,
|
|
|
|
PDiag(),
|
2009-10-11 17:03:14 +08:00
|
|
|
Target->getArgType(i), TargetLoc,
|
|
|
|
Source->getArgType(i), SourceLoc))
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool Sema::CheckExceptionSpecCompatibility(Expr *From, QualType ToType)
|
|
|
|
{
|
|
|
|
// First we check for applicability.
|
|
|
|
// Target type must be a function, function pointer or function reference.
|
|
|
|
const FunctionProtoType *ToFunc = GetUnderlyingFunction(ToType);
|
|
|
|
if (!ToFunc)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// SourceType must be a function or function pointer.
|
|
|
|
const FunctionProtoType *FromFunc = GetUnderlyingFunction(From->getType());
|
|
|
|
if (!FromFunc)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Now we've got the correct types on both sides, check their compatibility.
|
|
|
|
// This means that the source of the conversion can only throw a subset of
|
|
|
|
// the exceptions of the target, and any exception specs on arguments or
|
|
|
|
// return types must be equivalent.
|
2010-03-30 07:34:08 +08:00
|
|
|
return CheckExceptionSpecSubset(PDiag(diag::err_incompatible_exception_specs),
|
|
|
|
PDiag(), ToFunc,
|
|
|
|
From->getSourceRange().getBegin(),
|
2009-10-11 17:03:14 +08:00
|
|
|
FromFunc, SourceLocation());
|
|
|
|
}
|
|
|
|
|
|
|
|
bool Sema::CheckOverridingFunctionExceptionSpec(const CXXMethodDecl *New,
|
|
|
|
const CXXMethodDecl *Old) {
|
2011-05-20 13:57:18 +08:00
|
|
|
if (getLangOptions().CPlusPlus0x && isa<CXXDestructorDecl>(New)) {
|
|
|
|
// Don't check uninstantiated template destructors at all. We can only
|
|
|
|
// synthesize correct specs after the template is instantiated.
|
|
|
|
if (New->getParent()->isDependentType())
|
|
|
|
return false;
|
|
|
|
if (New->getParent()->isBeingDefined()) {
|
|
|
|
// The destructor might be updated once the definition is finished. So
|
|
|
|
// remember it and check later.
|
|
|
|
DelayedDestructorExceptionSpecChecks.push_back(std::make_pair(
|
|
|
|
cast<CXXDestructorDecl>(New), cast<CXXDestructorDecl>(Old)));
|
|
|
|
return false;
|
|
|
|
}
|
2011-05-19 13:13:44 +08:00
|
|
|
}
|
2011-05-24 10:11:43 +08:00
|
|
|
unsigned DiagID = diag::err_override_exception_spec;
|
2011-09-18 01:15:52 +08:00
|
|
|
if (getLangOptions().MicrosoftExt)
|
2011-05-24 10:11:43 +08:00
|
|
|
DiagID = diag::warn_override_exception_spec;
|
|
|
|
return CheckExceptionSpecSubset(PDiag(DiagID),
|
2010-03-30 07:34:08 +08:00
|
|
|
PDiag(diag::note_overridden_virtual_function),
|
2009-10-11 17:03:14 +08:00
|
|
|
Old->getType()->getAs<FunctionProtoType>(),
|
|
|
|
Old->getLocation(),
|
|
|
|
New->getType()->getAs<FunctionProtoType>(),
|
|
|
|
New->getLocation());
|
|
|
|
}
|
|
|
|
|
|
|
|
} // end namespace clang
|