blob: 8b6fcaa2b8029ab2a63c310b950aa61af0093615 [file] [log] [blame]
Sebastian Redldced2262009-10-11 09:03:14 +00001//===--- SemaExceptionSpec.cpp - C++ Exception Specifications ---*- C++ -*-===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file provides Sema routines for C++ exception specification testing.
11//
12//===----------------------------------------------------------------------===//
13
John McCall2d887082010-08-25 22:03:47 +000014#include "clang/Sema/SemaInternal.h"
Sebastian Redldced2262009-10-11 09:03:14 +000015#include "clang/AST/CXXInheritance.h"
16#include "clang/AST/Expr.h"
17#include "clang/AST/ExprCXX.h"
Douglas Gregor2eef8292010-03-24 07:14:45 +000018#include "clang/AST/TypeLoc.h"
19#include "clang/Lex/Preprocessor.h"
Douglas Gregore13ad832010-02-12 07:32:17 +000020#include "clang/Basic/Diagnostic.h"
21#include "clang/Basic/SourceManager.h"
Sebastian Redldced2262009-10-11 09:03:14 +000022#include "llvm/ADT/SmallPtrSet.h"
Benjamin Kramer8fe83e12012-02-04 13:45:25 +000023#include "llvm/ADT/SmallString.h"
Sebastian Redldced2262009-10-11 09:03:14 +000024
25namespace clang {
26
27static const FunctionProtoType *GetUnderlyingFunction(QualType T)
28{
29 if (const PointerType *PtrTy = T->getAs<PointerType>())
30 T = PtrTy->getPointeeType();
31 else if (const ReferenceType *RefTy = T->getAs<ReferenceType>())
32 T = RefTy->getPointeeType();
Sebastian Redlc3a3b7b2009-10-14 14:38:54 +000033 else if (const MemberPointerType *MPTy = T->getAs<MemberPointerType>())
34 T = MPTy->getPointeeType();
Sebastian Redldced2262009-10-11 09:03:14 +000035 return T->getAs<FunctionProtoType>();
36}
37
38/// CheckSpecifiedExceptionType - Check if the given type is valid in an
39/// exception specification. Incomplete types, or pointers to incomplete types
40/// other than void are not allowed.
41bool Sema::CheckSpecifiedExceptionType(QualType T, const SourceRange &Range) {
Sebastian Redldced2262009-10-11 09:03:14 +000042
Douglas Gregor0966f352009-12-10 18:13:52 +000043 // This check (and the similar one below) deals with issue 437, that changes
44 // C++ 9.2p2 this way:
45 // Within the class member-specification, the class is regarded as complete
46 // within function bodies, default arguments, exception-specifications, and
47 // constructor ctor-initializers (including such things in nested classes).
48 if (T->isRecordType() && T->getAs<RecordType>()->isBeingDefined())
49 return false;
50
Sebastian Redldced2262009-10-11 09:03:14 +000051 // C++ 15.4p2: A type denoted in an exception-specification shall not denote
52 // an incomplete type.
Sebastian Redl491b84c2009-10-14 14:59:48 +000053 if (RequireCompleteType(Range.getBegin(), T,
54 PDiag(diag::err_incomplete_in_exception_spec) << /*direct*/0 << Range))
55 return true;
Sebastian Redldced2262009-10-11 09:03:14 +000056
57 // C++ 15.4p2: A type denoted in an exception-specification shall not denote
58 // an incomplete type a pointer or reference to an incomplete type, other
59 // than (cv) void*.
60 int kind;
61 if (const PointerType* IT = T->getAs<PointerType>()) {
62 T = IT->getPointeeType();
63 kind = 1;
64 } else if (const ReferenceType* IT = T->getAs<ReferenceType>()) {
65 T = IT->getPointeeType();
66 kind = 2;
67 } else
68 return false;
69
Douglas Gregor0966f352009-12-10 18:13:52 +000070 // Again as before
71 if (T->isRecordType() && T->getAs<RecordType>()->isBeingDefined())
72 return false;
73
Sebastian Redl491b84c2009-10-14 14:59:48 +000074 if (!T->isVoidType() && RequireCompleteType(Range.getBegin(), T,
Douglas Gregor0966f352009-12-10 18:13:52 +000075 PDiag(diag::err_incomplete_in_exception_spec) << kind << Range))
Sebastian Redl491b84c2009-10-14 14:59:48 +000076 return true;
Sebastian Redldced2262009-10-11 09:03:14 +000077
78 return false;
79}
80
81/// CheckDistantExceptionSpec - Check if the given type is a pointer or pointer
82/// to member to a function with an exception specification. This means that
83/// it is invalid to add another level of indirection.
84bool Sema::CheckDistantExceptionSpec(QualType T) {
85 if (const PointerType *PT = T->getAs<PointerType>())
86 T = PT->getPointeeType();
87 else if (const MemberPointerType *PT = T->getAs<MemberPointerType>())
88 T = PT->getPointeeType();
89 else
90 return false;
91
92 const FunctionProtoType *FnT = T->getAs<FunctionProtoType>();
93 if (!FnT)
94 return false;
95
96 return FnT->hasExceptionSpec();
97}
98
Douglas Gregore13ad832010-02-12 07:32:17 +000099bool Sema::CheckEquivalentExceptionSpec(FunctionDecl *Old, FunctionDecl *New) {
Sebastian Redl99439d42011-03-15 19:52:30 +0000100 OverloadedOperatorKind OO = New->getDeclName().getCXXOverloadedOperator();
101 bool IsOperatorNew = OO == OO_New || OO == OO_Array_New;
Douglas Gregor2eef8292010-03-24 07:14:45 +0000102 bool MissingExceptionSpecification = false;
Douglas Gregore13ad832010-02-12 07:32:17 +0000103 bool MissingEmptyExceptionSpecification = false;
Francois Picheteedd4672011-03-19 23:05:18 +0000104 unsigned DiagID = diag::err_mismatched_exception_spec;
Francois Pichet62ec1f22011-09-17 17:15:52 +0000105 if (getLangOptions().MicrosoftExt)
Francois Pichetcf320c62011-04-22 08:25:24 +0000106 DiagID = diag::warn_mismatched_exception_spec;
Francois Picheteedd4672011-03-19 23:05:18 +0000107
108 if (!CheckEquivalentExceptionSpec(PDiag(DiagID),
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000109 PDiag(diag::note_previous_declaration),
Douglas Gregore13ad832010-02-12 07:32:17 +0000110 Old->getType()->getAs<FunctionProtoType>(),
111 Old->getLocation(),
112 New->getType()->getAs<FunctionProtoType>(),
113 New->getLocation(),
Douglas Gregor2eef8292010-03-24 07:14:45 +0000114 &MissingExceptionSpecification,
Sebastian Redl60618fa2011-03-12 11:50:43 +0000115 &MissingEmptyExceptionSpecification,
Sebastian Redl99439d42011-03-15 19:52:30 +0000116 /*AllowNoexceptAllMatchWithNoSpec=*/true,
117 IsOperatorNew))
Douglas Gregore13ad832010-02-12 07:32:17 +0000118 return false;
119
120 // The failure was something other than an empty exception
121 // specification; return an error.
Douglas Gregor2eef8292010-03-24 07:14:45 +0000122 if (!MissingExceptionSpecification && !MissingEmptyExceptionSpecification)
Douglas Gregore13ad832010-02-12 07:32:17 +0000123 return true;
124
John McCalle23cf432010-12-14 08:05:40 +0000125 const FunctionProtoType *NewProto
126 = New->getType()->getAs<FunctionProtoType>();
127
Douglas Gregore13ad832010-02-12 07:32:17 +0000128 // The new function declaration is only missing an empty exception
129 // specification "throw()". If the throw() specification came from a
130 // function in a system header that has C linkage, just add an empty
131 // exception specification to the "new" declaration. This is an
132 // egregious workaround for glibc, which adds throw() specifications
133 // to many libc functions as an optimization. Unfortunately, that
134 // optimization isn't permitted by the C++ standard, so we're forced
135 // to work around it here.
John McCalle23cf432010-12-14 08:05:40 +0000136 if (MissingEmptyExceptionSpecification && NewProto &&
Douglas Gregor2eef8292010-03-24 07:14:45 +0000137 (Old->getLocation().isInvalid() ||
138 Context.getSourceManager().isInSystemHeader(Old->getLocation())) &&
Douglas Gregore13ad832010-02-12 07:32:17 +0000139 Old->isExternC()) {
John McCalle23cf432010-12-14 08:05:40 +0000140 FunctionProtoType::ExtProtoInfo EPI = NewProto->getExtProtoInfo();
Sebastian Redl60618fa2011-03-12 11:50:43 +0000141 EPI.ExceptionSpecType = EST_DynamicNone;
Douglas Gregore13ad832010-02-12 07:32:17 +0000142 QualType NewType = Context.getFunctionType(NewProto->getResultType(),
143 NewProto->arg_type_begin(),
144 NewProto->getNumArgs(),
John McCalle23cf432010-12-14 08:05:40 +0000145 EPI);
Douglas Gregore13ad832010-02-12 07:32:17 +0000146 New->setType(NewType);
147 return false;
148 }
149
John McCalle23cf432010-12-14 08:05:40 +0000150 if (MissingExceptionSpecification && NewProto) {
Douglas Gregor2eef8292010-03-24 07:14:45 +0000151 const FunctionProtoType *OldProto
152 = Old->getType()->getAs<FunctionProtoType>();
153
John McCalle23cf432010-12-14 08:05:40 +0000154 FunctionProtoType::ExtProtoInfo EPI = NewProto->getExtProtoInfo();
Sebastian Redl60618fa2011-03-12 11:50:43 +0000155 EPI.ExceptionSpecType = OldProto->getExceptionSpecType();
156 if (EPI.ExceptionSpecType == EST_Dynamic) {
157 EPI.NumExceptions = OldProto->getNumExceptions();
158 EPI.Exceptions = OldProto->exception_begin();
159 } else if (EPI.ExceptionSpecType == EST_ComputedNoexcept) {
160 // FIXME: We can't just take the expression from the old prototype. It
161 // likely contains references to the old prototype's parameters.
162 }
John McCalle23cf432010-12-14 08:05:40 +0000163
Douglas Gregor2eef8292010-03-24 07:14:45 +0000164 // Update the type of the function with the appropriate exception
165 // specification.
166 QualType NewType = Context.getFunctionType(NewProto->getResultType(),
167 NewProto->arg_type_begin(),
168 NewProto->getNumArgs(),
John McCalle23cf432010-12-14 08:05:40 +0000169 EPI);
Douglas Gregor2eef8292010-03-24 07:14:45 +0000170 New->setType(NewType);
171
172 // If exceptions are disabled, suppress the warning about missing
173 // exception specifications for new and delete operators.
John McCall018591f2011-03-02 02:04:40 +0000174 if (!getLangOptions().CXXExceptions) {
Douglas Gregor2eef8292010-03-24 07:14:45 +0000175 switch (New->getDeclName().getCXXOverloadedOperator()) {
176 case OO_New:
177 case OO_Array_New:
178 case OO_Delete:
179 case OO_Array_Delete:
180 if (New->getDeclContext()->isTranslationUnit())
181 return false;
182 break;
183
184 default:
185 break;
186 }
187 }
188
189 // Warn about the lack of exception specification.
Dylan Noblesmithf7ccbad2012-02-05 02:13:05 +0000190 SmallString<128> ExceptionSpecString;
Douglas Gregor2eef8292010-03-24 07:14:45 +0000191 llvm::raw_svector_ostream OS(ExceptionSpecString);
Sebastian Redl60618fa2011-03-12 11:50:43 +0000192 switch (OldProto->getExceptionSpecType()) {
193 case EST_DynamicNone:
194 OS << "throw()";
195 break;
196
197 case EST_Dynamic: {
198 OS << "throw(";
199 bool OnFirstException = true;
200 for (FunctionProtoType::exception_iterator E = OldProto->exception_begin(),
201 EEnd = OldProto->exception_end();
202 E != EEnd;
203 ++E) {
204 if (OnFirstException)
205 OnFirstException = false;
206 else
207 OS << ", ";
208
Douglas Gregor8987b232011-09-27 23:30:47 +0000209 OS << E->getAsString(getPrintingPolicy());
Sebastian Redl60618fa2011-03-12 11:50:43 +0000210 }
211 OS << ")";
212 break;
Douglas Gregor2eef8292010-03-24 07:14:45 +0000213 }
Sebastian Redl60618fa2011-03-12 11:50:43 +0000214
215 case EST_BasicNoexcept:
216 OS << "noexcept";
217 break;
218
219 case EST_ComputedNoexcept:
220 OS << "noexcept(";
Douglas Gregor8987b232011-09-27 23:30:47 +0000221 OldProto->getNoexceptExpr()->printPretty(OS, Context, 0,
222 getPrintingPolicy());
Sebastian Redl60618fa2011-03-12 11:50:43 +0000223 OS << ")";
224 break;
225
226 default:
David Blaikieb219cfc2011-09-23 05:06:16 +0000227 llvm_unreachable("This spec type is compatible with none.");
Sebastian Redl60618fa2011-03-12 11:50:43 +0000228 }
Douglas Gregor2eef8292010-03-24 07:14:45 +0000229 OS.flush();
230
Abramo Bagnara796aa442011-03-12 11:17:06 +0000231 SourceLocation FixItLoc;
Douglas Gregor2eef8292010-03-24 07:14:45 +0000232 if (TypeSourceInfo *TSInfo = New->getTypeSourceInfo()) {
Abramo Bagnara723df242010-12-14 22:11:44 +0000233 TypeLoc TL = TSInfo->getTypeLoc().IgnoreParens();
Douglas Gregor2eef8292010-03-24 07:14:45 +0000234 if (const FunctionTypeLoc *FTLoc = dyn_cast<FunctionTypeLoc>(&TL))
Abramo Bagnara796aa442011-03-12 11:17:06 +0000235 FixItLoc = PP.getLocForEndOfToken(FTLoc->getLocalRangeEnd());
Douglas Gregor2eef8292010-03-24 07:14:45 +0000236 }
237
Abramo Bagnara796aa442011-03-12 11:17:06 +0000238 if (FixItLoc.isInvalid())
Douglas Gregor2eef8292010-03-24 07:14:45 +0000239 Diag(New->getLocation(), diag::warn_missing_exception_specification)
240 << New << OS.str();
241 else {
242 // FIXME: This will get more complicated with C++0x
243 // late-specified return types.
244 Diag(New->getLocation(), diag::warn_missing_exception_specification)
245 << New << OS.str()
Abramo Bagnara796aa442011-03-12 11:17:06 +0000246 << FixItHint::CreateInsertion(FixItLoc, " " + OS.str().str());
Douglas Gregor2eef8292010-03-24 07:14:45 +0000247 }
248
249 if (!Old->getLocation().isInvalid())
250 Diag(Old->getLocation(), diag::note_previous_declaration);
251
252 return false;
253 }
254
Francois Picheteedd4672011-03-19 23:05:18 +0000255 Diag(New->getLocation(), DiagID);
Douglas Gregore13ad832010-02-12 07:32:17 +0000256 Diag(Old->getLocation(), diag::note_previous_declaration);
257 return true;
258}
259
Sebastian Redldced2262009-10-11 09:03:14 +0000260/// CheckEquivalentExceptionSpec - Check if the two types have equivalent
261/// exception specifications. Exception specifications are equivalent if
262/// they allow exactly the same set of exception types. It does not matter how
263/// that is achieved. See C++ [except.spec]p2.
264bool Sema::CheckEquivalentExceptionSpec(
265 const FunctionProtoType *Old, SourceLocation OldLoc,
266 const FunctionProtoType *New, SourceLocation NewLoc) {
Francois Picheteedd4672011-03-19 23:05:18 +0000267 unsigned DiagID = diag::err_mismatched_exception_spec;
Francois Pichet62ec1f22011-09-17 17:15:52 +0000268 if (getLangOptions().MicrosoftExt)
Francois Pichetcf320c62011-04-22 08:25:24 +0000269 DiagID = diag::warn_mismatched_exception_spec;
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000270 return CheckEquivalentExceptionSpec(
Francois Picheteedd4672011-03-19 23:05:18 +0000271 PDiag(DiagID),
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000272 PDiag(diag::note_previous_declaration),
Sebastian Redldced2262009-10-11 09:03:14 +0000273 Old, OldLoc, New, NewLoc);
274}
275
Sebastian Redl60618fa2011-03-12 11:50:43 +0000276/// CheckEquivalentExceptionSpec - Check if the two types have compatible
277/// exception specifications. See C++ [except.spec]p3.
278bool Sema::CheckEquivalentExceptionSpec(const PartialDiagnostic &DiagID,
Douglas Gregor2eef8292010-03-24 07:14:45 +0000279 const PartialDiagnostic & NoteID,
Sebastian Redl60618fa2011-03-12 11:50:43 +0000280 const FunctionProtoType *Old,
Douglas Gregor2eef8292010-03-24 07:14:45 +0000281 SourceLocation OldLoc,
Sebastian Redl60618fa2011-03-12 11:50:43 +0000282 const FunctionProtoType *New,
Douglas Gregor2eef8292010-03-24 07:14:45 +0000283 SourceLocation NewLoc,
284 bool *MissingExceptionSpecification,
Sebastian Redl60618fa2011-03-12 11:50:43 +0000285 bool*MissingEmptyExceptionSpecification,
Sebastian Redl99439d42011-03-15 19:52:30 +0000286 bool AllowNoexceptAllMatchWithNoSpec,
287 bool IsOperatorNew) {
John McCall811d0be2010-05-28 08:37:35 +0000288 // Just completely ignore this under -fno-exceptions.
John McCall018591f2011-03-02 02:04:40 +0000289 if (!getLangOptions().CXXExceptions)
John McCall811d0be2010-05-28 08:37:35 +0000290 return false;
291
Douglas Gregor2eef8292010-03-24 07:14:45 +0000292 if (MissingExceptionSpecification)
293 *MissingExceptionSpecification = false;
294
Douglas Gregore13ad832010-02-12 07:32:17 +0000295 if (MissingEmptyExceptionSpecification)
296 *MissingEmptyExceptionSpecification = false;
297
Sebastian Redl60618fa2011-03-12 11:50:43 +0000298 // C++0x [except.spec]p3: Two exception-specifications are compatible if:
299 // - both are non-throwing, regardless of their form,
300 // - both have the form noexcept(constant-expression) and the constant-
301 // expressions are equivalent,
Sebastian Redl60618fa2011-03-12 11:50:43 +0000302 // - both are dynamic-exception-specifications that have the same set of
303 // adjusted types.
304 //
305 // C++0x [except.spec]p12: An exception-specifcation is non-throwing if it is
306 // of the form throw(), noexcept, or noexcept(constant-expression) where the
307 // constant-expression yields true.
308 //
Sebastian Redl60618fa2011-03-12 11:50:43 +0000309 // C++0x [except.spec]p4: If any declaration of a function has an exception-
310 // specifier that is not a noexcept-specification allowing all exceptions,
311 // all declarations [...] of that function shall have a compatible
312 // exception-specification.
313 //
314 // That last point basically means that noexcept(false) matches no spec.
315 // It's considered when AllowNoexceptAllMatchWithNoSpec is true.
316
317 ExceptionSpecificationType OldEST = Old->getExceptionSpecType();
318 ExceptionSpecificationType NewEST = New->getExceptionSpecType();
319
Richard Smith7a614d82011-06-11 17:19:42 +0000320 assert(OldEST != EST_Delayed && NewEST != EST_Delayed &&
321 "Shouldn't see unknown exception specifications here");
322
Sebastian Redl60618fa2011-03-12 11:50:43 +0000323 // Shortcut the case where both have no spec.
324 if (OldEST == EST_None && NewEST == EST_None)
325 return false;
326
Sebastian Redl8026f6d2011-03-13 17:09:40 +0000327 FunctionProtoType::NoexceptResult OldNR = Old->getNoexceptSpec(Context);
328 FunctionProtoType::NoexceptResult NewNR = New->getNoexceptSpec(Context);
Sebastian Redl60618fa2011-03-12 11:50:43 +0000329 if (OldNR == FunctionProtoType::NR_BadNoexcept ||
330 NewNR == FunctionProtoType::NR_BadNoexcept)
331 return false;
332
333 // Dependent noexcept specifiers are compatible with each other, but nothing
334 // else.
335 // One noexcept is compatible with another if the argument is the same
336 if (OldNR == NewNR &&
337 OldNR != FunctionProtoType::NR_NoNoexcept &&
338 NewNR != FunctionProtoType::NR_NoNoexcept)
339 return false;
340 if (OldNR != NewNR &&
341 OldNR != FunctionProtoType::NR_NoNoexcept &&
342 NewNR != FunctionProtoType::NR_NoNoexcept) {
343 Diag(NewLoc, DiagID);
344 if (NoteID.getDiagID() != 0)
345 Diag(OldLoc, NoteID);
346 return true;
Douglas Gregor5b6f7692010-08-30 15:04:51 +0000347 }
348
Sebastian Redl60618fa2011-03-12 11:50:43 +0000349 // The MS extension throw(...) is compatible with itself.
350 if (OldEST == EST_MSAny && NewEST == EST_MSAny)
Sebastian Redldced2262009-10-11 09:03:14 +0000351 return false;
Sebastian Redl60618fa2011-03-12 11:50:43 +0000352
353 // It's also compatible with no spec.
354 if ((OldEST == EST_None && NewEST == EST_MSAny) ||
355 (OldEST == EST_MSAny && NewEST == EST_None))
356 return false;
357
358 // It's also compatible with noexcept(false).
359 if (OldEST == EST_MSAny && NewNR == FunctionProtoType::NR_Throw)
360 return false;
361 if (NewEST == EST_MSAny && OldNR == FunctionProtoType::NR_Throw)
362 return false;
363
364 // As described above, noexcept(false) matches no spec only for functions.
365 if (AllowNoexceptAllMatchWithNoSpec) {
366 if (OldEST == EST_None && NewNR == FunctionProtoType::NR_Throw)
367 return false;
368 if (NewEST == EST_None && OldNR == FunctionProtoType::NR_Throw)
369 return false;
370 }
371
372 // Any non-throwing specifications are compatible.
373 bool OldNonThrowing = OldNR == FunctionProtoType::NR_Nothrow ||
374 OldEST == EST_DynamicNone;
375 bool NewNonThrowing = NewNR == FunctionProtoType::NR_Nothrow ||
376 NewEST == EST_DynamicNone;
377 if (OldNonThrowing && NewNonThrowing)
378 return false;
379
Sebastian Redl99439d42011-03-15 19:52:30 +0000380 // As a special compatibility feature, under C++0x we accept no spec and
381 // throw(std::bad_alloc) as equivalent for operator new and operator new[].
382 // This is because the implicit declaration changed, but old code would break.
383 if (getLangOptions().CPlusPlus0x && IsOperatorNew) {
384 const FunctionProtoType *WithExceptions = 0;
385 if (OldEST == EST_None && NewEST == EST_Dynamic)
386 WithExceptions = New;
387 else if (OldEST == EST_Dynamic && NewEST == EST_None)
388 WithExceptions = Old;
389 if (WithExceptions && WithExceptions->getNumExceptions() == 1) {
390 // One has no spec, the other throw(something). If that something is
391 // std::bad_alloc, all conditions are met.
392 QualType Exception = *WithExceptions->exception_begin();
393 if (CXXRecordDecl *ExRecord = Exception->getAsCXXRecordDecl()) {
394 IdentifierInfo* Name = ExRecord->getIdentifier();
395 if (Name && Name->getName() == "bad_alloc") {
396 // It's called bad_alloc, but is it in std?
397 DeclContext* DC = ExRecord->getDeclContext();
Sebastian Redld8f2e8e2011-03-15 20:41:09 +0000398 DC = DC->getEnclosingNamespaceContext();
399 if (NamespaceDecl* NS = dyn_cast<NamespaceDecl>(DC)) {
Sebastian Redl99439d42011-03-15 19:52:30 +0000400 IdentifierInfo* NSName = NS->getIdentifier();
Sebastian Redld8f2e8e2011-03-15 20:41:09 +0000401 DC = DC->getParent();
Sebastian Redl99439d42011-03-15 19:52:30 +0000402 if (NSName && NSName->getName() == "std" &&
Sebastian Redld8f2e8e2011-03-15 20:41:09 +0000403 DC->getEnclosingNamespaceContext()->isTranslationUnit()) {
Sebastian Redl99439d42011-03-15 19:52:30 +0000404 return false;
Sebastian Redld8f2e8e2011-03-15 20:41:09 +0000405 }
Sebastian Redl99439d42011-03-15 19:52:30 +0000406 }
407 }
408 }
409 }
410 }
411
Sebastian Redl60618fa2011-03-12 11:50:43 +0000412 // At this point, the only remaining valid case is two matching dynamic
413 // specifications. We return here unless both specifications are dynamic.
414 if (OldEST != EST_Dynamic || NewEST != EST_Dynamic) {
Douglas Gregor2eef8292010-03-24 07:14:45 +0000415 if (MissingExceptionSpecification && Old->hasExceptionSpec() &&
Douglas Gregore13ad832010-02-12 07:32:17 +0000416 !New->hasExceptionSpec()) {
Douglas Gregor2eef8292010-03-24 07:14:45 +0000417 // The old type has an exception specification of some sort, but
418 // the new type does not.
419 *MissingExceptionSpecification = true;
420
Sebastian Redl60618fa2011-03-12 11:50:43 +0000421 if (MissingEmptyExceptionSpecification && OldNonThrowing) {
422 // The old type has a throw() or noexcept(true) exception specification
423 // and the new type has no exception specification, and the caller asked
Douglas Gregor2eef8292010-03-24 07:14:45 +0000424 // to handle this itself.
425 *MissingEmptyExceptionSpecification = true;
426 }
427
Douglas Gregore13ad832010-02-12 07:32:17 +0000428 return true;
429 }
430
Sebastian Redldced2262009-10-11 09:03:14 +0000431 Diag(NewLoc, DiagID);
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000432 if (NoteID.getDiagID() != 0)
Sebastian Redldced2262009-10-11 09:03:14 +0000433 Diag(OldLoc, NoteID);
434 return true;
435 }
436
Sebastian Redl60618fa2011-03-12 11:50:43 +0000437 assert(OldEST == EST_Dynamic && NewEST == EST_Dynamic &&
438 "Exception compatibility logic error: non-dynamic spec slipped through.");
439
Sebastian Redldced2262009-10-11 09:03:14 +0000440 bool Success = true;
Sebastian Redl60618fa2011-03-12 11:50:43 +0000441 // Both have a dynamic exception spec. Collect the first set, then compare
Sebastian Redldced2262009-10-11 09:03:14 +0000442 // to the second.
Sebastian Redl1219d152009-10-14 15:06:25 +0000443 llvm::SmallPtrSet<CanQualType, 8> OldTypes, NewTypes;
Sebastian Redldced2262009-10-11 09:03:14 +0000444 for (FunctionProtoType::exception_iterator I = Old->exception_begin(),
445 E = Old->exception_end(); I != E; ++I)
Sebastian Redl1219d152009-10-14 15:06:25 +0000446 OldTypes.insert(Context.getCanonicalType(*I).getUnqualifiedType());
Sebastian Redldced2262009-10-11 09:03:14 +0000447
448 for (FunctionProtoType::exception_iterator I = New->exception_begin(),
Sebastian Redl5db4d902009-10-11 09:11:23 +0000449 E = New->exception_end(); I != E && Success; ++I) {
Sebastian Redl1219d152009-10-14 15:06:25 +0000450 CanQualType TypePtr = Context.getCanonicalType(*I).getUnqualifiedType();
Sebastian Redl5db4d902009-10-11 09:11:23 +0000451 if(OldTypes.count(TypePtr))
452 NewTypes.insert(TypePtr);
453 else
454 Success = false;
455 }
Sebastian Redldced2262009-10-11 09:03:14 +0000456
Sebastian Redl5db4d902009-10-11 09:11:23 +0000457 Success = Success && OldTypes.size() == NewTypes.size();
Sebastian Redldced2262009-10-11 09:03:14 +0000458
459 if (Success) {
460 return false;
461 }
462 Diag(NewLoc, DiagID);
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000463 if (NoteID.getDiagID() != 0)
Sebastian Redldced2262009-10-11 09:03:14 +0000464 Diag(OldLoc, NoteID);
465 return true;
466}
467
468/// CheckExceptionSpecSubset - Check whether the second function type's
469/// exception specification is a subset (or equivalent) of the first function
470/// type. This is used by override and pointer assignment checks.
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000471bool Sema::CheckExceptionSpecSubset(
472 const PartialDiagnostic &DiagID, const PartialDiagnostic & NoteID,
Sebastian Redldced2262009-10-11 09:03:14 +0000473 const FunctionProtoType *Superset, SourceLocation SuperLoc,
474 const FunctionProtoType *Subset, SourceLocation SubLoc) {
John McCall811d0be2010-05-28 08:37:35 +0000475
476 // Just auto-succeed under -fno-exceptions.
John McCall018591f2011-03-02 02:04:40 +0000477 if (!getLangOptions().CXXExceptions)
John McCall811d0be2010-05-28 08:37:35 +0000478 return false;
479
Sebastian Redldced2262009-10-11 09:03:14 +0000480 // FIXME: As usual, we could be more specific in our error messages, but
481 // that better waits until we've got types with source locations.
482
483 if (!SubLoc.isValid())
484 SubLoc = SuperLoc;
485
Sebastian Redl60618fa2011-03-12 11:50:43 +0000486 ExceptionSpecificationType SuperEST = Superset->getExceptionSpecType();
487
Sebastian Redldced2262009-10-11 09:03:14 +0000488 // If superset contains everything, we're done.
Sebastian Redl60618fa2011-03-12 11:50:43 +0000489 if (SuperEST == EST_None || SuperEST == EST_MSAny)
Sebastian Redldced2262009-10-11 09:03:14 +0000490 return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
491
Sebastian Redl60618fa2011-03-12 11:50:43 +0000492 // If there are dependent noexcept specs, assume everything is fine. Unlike
493 // with the equivalency check, this is safe in this case, because we don't
494 // want to merge declarations. Checks after instantiation will catch any
495 // omissions we make here.
496 // We also shortcut checking if a noexcept expression was bad.
497
Sebastian Redl8026f6d2011-03-13 17:09:40 +0000498 FunctionProtoType::NoexceptResult SuperNR =Superset->getNoexceptSpec(Context);
Sebastian Redl60618fa2011-03-12 11:50:43 +0000499 if (SuperNR == FunctionProtoType::NR_BadNoexcept ||
500 SuperNR == FunctionProtoType::NR_Dependent)
501 return false;
502
503 // Another case of the superset containing everything.
504 if (SuperNR == FunctionProtoType::NR_Throw)
505 return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
506
507 ExceptionSpecificationType SubEST = Subset->getExceptionSpecType();
508
Richard Smith7a614d82011-06-11 17:19:42 +0000509 assert(SuperEST != EST_Delayed && SubEST != EST_Delayed &&
510 "Shouldn't see unknown exception specifications here");
511
Sebastian Redldced2262009-10-11 09:03:14 +0000512 // It does not. If the subset contains everything, we've failed.
Sebastian Redl60618fa2011-03-12 11:50:43 +0000513 if (SubEST == EST_None || SubEST == EST_MSAny) {
Sebastian Redldced2262009-10-11 09:03:14 +0000514 Diag(SubLoc, DiagID);
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000515 if (NoteID.getDiagID() != 0)
Sebastian Redldced2262009-10-11 09:03:14 +0000516 Diag(SuperLoc, NoteID);
517 return true;
518 }
519
Sebastian Redl8026f6d2011-03-13 17:09:40 +0000520 FunctionProtoType::NoexceptResult SubNR = Subset->getNoexceptSpec(Context);
Sebastian Redl60618fa2011-03-12 11:50:43 +0000521 if (SubNR == FunctionProtoType::NR_BadNoexcept ||
522 SubNR == FunctionProtoType::NR_Dependent)
523 return false;
524
525 // Another case of the subset containing everything.
526 if (SubNR == FunctionProtoType::NR_Throw) {
527 Diag(SubLoc, DiagID);
528 if (NoteID.getDiagID() != 0)
529 Diag(SuperLoc, NoteID);
530 return true;
531 }
532
533 // If the subset contains nothing, we're done.
534 if (SubEST == EST_DynamicNone || SubNR == FunctionProtoType::NR_Nothrow)
535 return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
536
537 // Otherwise, if the superset contains nothing, we've failed.
538 if (SuperEST == EST_DynamicNone || SuperNR == FunctionProtoType::NR_Nothrow) {
539 Diag(SubLoc, DiagID);
540 if (NoteID.getDiagID() != 0)
541 Diag(SuperLoc, NoteID);
542 return true;
543 }
544
545 assert(SuperEST == EST_Dynamic && SubEST == EST_Dynamic &&
546 "Exception spec subset: non-dynamic case slipped through.");
547
548 // Neither contains everything or nothing. Do a proper comparison.
Sebastian Redldced2262009-10-11 09:03:14 +0000549 for (FunctionProtoType::exception_iterator SubI = Subset->exception_begin(),
550 SubE = Subset->exception_end(); SubI != SubE; ++SubI) {
551 // Take one type from the subset.
552 QualType CanonicalSubT = Context.getCanonicalType(*SubI);
Sebastian Redlc3a3b7b2009-10-14 14:38:54 +0000553 // Unwrap pointers and references so that we can do checks within a class
554 // hierarchy. Don't unwrap member pointers; they don't have hierarchy
555 // conversions on the pointee.
Sebastian Redldced2262009-10-11 09:03:14 +0000556 bool SubIsPointer = false;
557 if (const ReferenceType *RefTy = CanonicalSubT->getAs<ReferenceType>())
558 CanonicalSubT = RefTy->getPointeeType();
559 if (const PointerType *PtrTy = CanonicalSubT->getAs<PointerType>()) {
560 CanonicalSubT = PtrTy->getPointeeType();
561 SubIsPointer = true;
562 }
563 bool SubIsClass = CanonicalSubT->isRecordType();
Douglas Gregora4923eb2009-11-16 21:35:15 +0000564 CanonicalSubT = CanonicalSubT.getLocalUnqualifiedType();
Sebastian Redldced2262009-10-11 09:03:14 +0000565
566 CXXBasePaths Paths(/*FindAmbiguities=*/true, /*RecordPaths=*/true,
567 /*DetectVirtual=*/false);
568
569 bool Contained = false;
570 // Make sure it's in the superset.
571 for (FunctionProtoType::exception_iterator SuperI =
572 Superset->exception_begin(), SuperE = Superset->exception_end();
573 SuperI != SuperE; ++SuperI) {
574 QualType CanonicalSuperT = Context.getCanonicalType(*SuperI);
575 // SubT must be SuperT or derived from it, or pointer or reference to
576 // such types.
577 if (const ReferenceType *RefTy = CanonicalSuperT->getAs<ReferenceType>())
578 CanonicalSuperT = RefTy->getPointeeType();
579 if (SubIsPointer) {
580 if (const PointerType *PtrTy = CanonicalSuperT->getAs<PointerType>())
581 CanonicalSuperT = PtrTy->getPointeeType();
582 else {
583 continue;
584 }
585 }
Douglas Gregora4923eb2009-11-16 21:35:15 +0000586 CanonicalSuperT = CanonicalSuperT.getLocalUnqualifiedType();
Sebastian Redldced2262009-10-11 09:03:14 +0000587 // If the types are the same, move on to the next type in the subset.
588 if (CanonicalSubT == CanonicalSuperT) {
589 Contained = true;
590 break;
591 }
592
593 // Otherwise we need to check the inheritance.
594 if (!SubIsClass || !CanonicalSuperT->isRecordType())
595 continue;
596
597 Paths.clear();
598 if (!IsDerivedFrom(CanonicalSubT, CanonicalSuperT, Paths))
599 continue;
600
Douglas Gregore0d5fe22010-05-21 20:29:55 +0000601 if (Paths.isAmbiguous(Context.getCanonicalType(CanonicalSuperT)))
Sebastian Redldced2262009-10-11 09:03:14 +0000602 continue;
603
John McCall6b2accb2010-02-10 09:31:12 +0000604 // Do this check from a context without privileges.
John McCall58e6f342010-03-16 05:22:47 +0000605 switch (CheckBaseClassAccess(SourceLocation(),
John McCall6b2accb2010-02-10 09:31:12 +0000606 CanonicalSuperT, CanonicalSubT,
607 Paths.front(),
John McCall58e6f342010-03-16 05:22:47 +0000608 /*Diagnostic*/ 0,
John McCall6b2accb2010-02-10 09:31:12 +0000609 /*ForceCheck*/ true,
John McCall58e6f342010-03-16 05:22:47 +0000610 /*ForceUnprivileged*/ true)) {
John McCall6b2accb2010-02-10 09:31:12 +0000611 case AR_accessible: break;
612 case AR_inaccessible: continue;
613 case AR_dependent:
614 llvm_unreachable("access check dependent for unprivileged context");
John McCall6b2accb2010-02-10 09:31:12 +0000615 case AR_delayed:
616 llvm_unreachable("access check delayed in non-declaration");
John McCall6b2accb2010-02-10 09:31:12 +0000617 }
Sebastian Redldced2262009-10-11 09:03:14 +0000618
619 Contained = true;
620 break;
621 }
622 if (!Contained) {
623 Diag(SubLoc, DiagID);
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000624 if (NoteID.getDiagID() != 0)
Sebastian Redldced2262009-10-11 09:03:14 +0000625 Diag(SuperLoc, NoteID);
626 return true;
627 }
628 }
629 // We've run half the gauntlet.
630 return CheckParamExceptionSpec(NoteID, Superset, SuperLoc, Subset, SubLoc);
631}
632
633static bool CheckSpecForTypesEquivalent(Sema &S,
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000634 const PartialDiagnostic &DiagID, const PartialDiagnostic & NoteID,
Sebastian Redldced2262009-10-11 09:03:14 +0000635 QualType Target, SourceLocation TargetLoc,
636 QualType Source, SourceLocation SourceLoc)
637{
638 const FunctionProtoType *TFunc = GetUnderlyingFunction(Target);
639 if (!TFunc)
640 return false;
641 const FunctionProtoType *SFunc = GetUnderlyingFunction(Source);
642 if (!SFunc)
643 return false;
644
645 return S.CheckEquivalentExceptionSpec(DiagID, NoteID, TFunc, TargetLoc,
646 SFunc, SourceLoc);
647}
648
649/// CheckParamExceptionSpec - Check if the parameter and return types of the
650/// two functions have equivalent exception specs. This is part of the
651/// assignment and override compatibility check. We do not check the parameters
652/// of parameter function pointers recursively, as no sane programmer would
653/// even be able to write such a function type.
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000654bool Sema::CheckParamExceptionSpec(const PartialDiagnostic & NoteID,
Sebastian Redldced2262009-10-11 09:03:14 +0000655 const FunctionProtoType *Target, SourceLocation TargetLoc,
656 const FunctionProtoType *Source, SourceLocation SourceLoc)
657{
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000658 if (CheckSpecForTypesEquivalent(*this,
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000659 PDiag(diag::err_deep_exception_specs_differ) << 0,
660 PDiag(),
Sebastian Redldced2262009-10-11 09:03:14 +0000661 Target->getResultType(), TargetLoc,
662 Source->getResultType(), SourceLoc))
663 return true;
664
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000665 // We shouldn't even be testing this unless the arguments are otherwise
Sebastian Redldced2262009-10-11 09:03:14 +0000666 // compatible.
667 assert(Target->getNumArgs() == Source->getNumArgs() &&
668 "Functions have different argument counts.");
669 for (unsigned i = 0, E = Target->getNumArgs(); i != E; ++i) {
Sebastian Redl37c38ec2009-10-14 16:09:29 +0000670 if (CheckSpecForTypesEquivalent(*this,
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000671 PDiag(diag::err_deep_exception_specs_differ) << 1,
672 PDiag(),
Sebastian Redldced2262009-10-11 09:03:14 +0000673 Target->getArgType(i), TargetLoc,
674 Source->getArgType(i), SourceLoc))
675 return true;
676 }
677 return false;
678}
679
680bool Sema::CheckExceptionSpecCompatibility(Expr *From, QualType ToType)
681{
682 // First we check for applicability.
683 // Target type must be a function, function pointer or function reference.
684 const FunctionProtoType *ToFunc = GetUnderlyingFunction(ToType);
685 if (!ToFunc)
686 return false;
687
688 // SourceType must be a function or function pointer.
689 const FunctionProtoType *FromFunc = GetUnderlyingFunction(From->getType());
690 if (!FromFunc)
691 return false;
692
693 // Now we've got the correct types on both sides, check their compatibility.
694 // This means that the source of the conversion can only throw a subset of
695 // the exceptions of the target, and any exception specs on arguments or
696 // return types must be equivalent.
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000697 return CheckExceptionSpecSubset(PDiag(diag::err_incompatible_exception_specs),
698 PDiag(), ToFunc,
699 From->getSourceRange().getBegin(),
Sebastian Redldced2262009-10-11 09:03:14 +0000700 FromFunc, SourceLocation());
701}
702
703bool Sema::CheckOverridingFunctionExceptionSpec(const CXXMethodDecl *New,
704 const CXXMethodDecl *Old) {
Sebastian Redla0448262011-05-20 05:57:18 +0000705 if (getLangOptions().CPlusPlus0x && isa<CXXDestructorDecl>(New)) {
706 // Don't check uninstantiated template destructors at all. We can only
707 // synthesize correct specs after the template is instantiated.
708 if (New->getParent()->isDependentType())
709 return false;
710 if (New->getParent()->isBeingDefined()) {
711 // The destructor might be updated once the definition is finished. So
712 // remember it and check later.
713 DelayedDestructorExceptionSpecChecks.push_back(std::make_pair(
714 cast<CXXDestructorDecl>(New), cast<CXXDestructorDecl>(Old)));
715 return false;
716 }
Sebastian Redl0ee33912011-05-19 05:13:44 +0000717 }
Francois Pichet0f161592011-05-24 02:11:43 +0000718 unsigned DiagID = diag::err_override_exception_spec;
Francois Pichet62ec1f22011-09-17 17:15:52 +0000719 if (getLangOptions().MicrosoftExt)
Francois Pichet0f161592011-05-24 02:11:43 +0000720 DiagID = diag::warn_override_exception_spec;
721 return CheckExceptionSpecSubset(PDiag(DiagID),
Douglas Gregorfe6b2d42010-03-29 23:34:08 +0000722 PDiag(diag::note_overridden_virtual_function),
Sebastian Redldced2262009-10-11 09:03:14 +0000723 Old->getType()->getAs<FunctionProtoType>(),
724 Old->getLocation(),
725 New->getType()->getAs<FunctionProtoType>(),
726 New->getLocation());
727}
728
729} // end namespace clang