2006-11-10 13:07:45 +08:00
|
|
|
//===--- SemaStmt.cpp - Semantic Analysis for Statements ------------------===//
|
|
|
|
//
|
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-30 03:59:25 +08:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2006-11-10 13:07:45 +08:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
//
|
|
|
|
// This file implements semantic analysis for statements.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2010-08-26 06:03:47 +08:00
|
|
|
#include "clang/Sema/SemaInternal.h"
|
2007-08-23 13:46:52 +08:00
|
|
|
#include "clang/AST/ASTContext.h"
|
2013-01-10 07:04:56 +08:00
|
|
|
#include "clang/AST/ASTDiagnostic.h"
|
2011-11-11 11:57:31 +08:00
|
|
|
#include "clang/AST/CharUnits.h"
|
2008-08-11 13:35:13 +08:00
|
|
|
#include "clang/AST/DeclObjC.h"
|
2012-05-01 02:01:30 +08:00
|
|
|
#include "clang/AST/EvaluatedExprVisitor.h"
|
2009-11-23 21:46:08 +08:00
|
|
|
#include "clang/AST/ExprCXX.h"
|
2009-08-17 00:57:27 +08:00
|
|
|
#include "clang/AST/ExprObjC.h"
|
2009-04-26 09:32:48 +08:00
|
|
|
#include "clang/AST/StmtCXX.h"
|
2012-12-04 17:13:33 +08:00
|
|
|
#include "clang/AST/StmtObjC.h"
|
2010-04-07 06:24:14 +08:00
|
|
|
#include "clang/AST/TypeLoc.h"
|
2007-11-25 08:25:21 +08:00
|
|
|
#include "clang/Basic/TargetInfo.h"
|
2012-12-04 17:13:33 +08:00
|
|
|
#include "clang/Lex/Preprocessor.h"
|
|
|
|
#include "clang/Sema/Initialization.h"
|
|
|
|
#include "clang/Sema/Lookup.h"
|
|
|
|
#include "clang/Sema/Scope.h"
|
|
|
|
#include "clang/Sema/ScopeInfo.h"
|
2011-02-22 05:40:33 +08:00
|
|
|
#include "llvm/ADT/ArrayRef.h"
|
2009-07-30 01:15:45 +08:00
|
|
|
#include "llvm/ADT/STLExtras.h"
|
2012-05-01 02:01:30 +08:00
|
|
|
#include "llvm/ADT/SmallPtrSet.h"
|
2012-05-17 00:11:17 +08:00
|
|
|
#include "llvm/ADT/SmallString.h"
|
2009-07-30 01:15:45 +08:00
|
|
|
#include "llvm/ADT/SmallVector.h"
|
2006-11-10 13:07:45 +08:00
|
|
|
using namespace clang;
|
2010-08-25 16:40:02 +08:00
|
|
|
using namespace sema;
|
2006-11-10 13:07:45 +08:00
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
StmtResult Sema::ActOnExprStmt(ExprResult FE) {
|
|
|
|
if (FE.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
FE = ActOnFinishFullExpr(FE.get(), FE.get()->getExprLoc(),
|
|
|
|
/*DiscardedValue*/ true);
|
|
|
|
if (FE.isInvalid())
|
2010-12-15 09:34:56 +08:00
|
|
|
return StmtError();
|
|
|
|
|
2008-07-26 07:18:17 +08:00
|
|
|
// C99 6.8.3p2: The expression in an expression statement is evaluated as a
|
|
|
|
// void expression for its side effects. Conversion to void allows any
|
|
|
|
// operand, even incomplete types.
|
2008-12-21 20:04:03 +08:00
|
|
|
|
2008-07-26 07:18:17 +08:00
|
|
|
// Same thing in for stmt first clause (when expr) and third clause.
|
2013-01-15 06:39:08 +08:00
|
|
|
return Owned(static_cast<Stmt*>(FE.take()));
|
2007-06-27 13:38:08 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2013-03-22 10:10:40 +08:00
|
|
|
StmtResult Sema::ActOnExprStmtError() {
|
|
|
|
DiscardCleanupsInEvaluationContext();
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
2011-04-27 13:04:02 +08:00
|
|
|
StmtResult Sema::ActOnNullStmt(SourceLocation SemiLoc,
|
2011-09-02 05:53:45 +08:00
|
|
|
bool HasLeadingEmptyMacro) {
|
|
|
|
return Owned(new (Context) NullStmt(SemiLoc, HasLeadingEmptyMacro));
|
2007-05-28 09:45:28 +08:00
|
|
|
}
|
|
|
|
|
2011-02-18 09:27:55 +08:00
|
|
|
StmtResult Sema::ActOnDeclStmt(DeclGroupPtrTy dg, SourceLocation StartLoc,
|
|
|
|
SourceLocation EndLoc) {
|
2009-04-13 04:13:14 +08:00
|
|
|
DeclGroupRef DG = dg.getAsVal<DeclGroupRef>();
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-03-30 00:50:03 +08:00
|
|
|
// If we have an invalid decl, just return an error.
|
2009-04-13 04:13:14 +08:00
|
|
|
if (DG.isNull()) return StmtError();
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-03-04 12:23:07 +08:00
|
|
|
return Owned(new (Context) DeclStmt(DG, StartLoc, EndLoc));
|
2007-05-30 06:59:26 +08:00
|
|
|
}
|
2006-11-10 13:07:45 +08:00
|
|
|
|
2009-11-20 06:12:37 +08:00
|
|
|
void Sema::ActOnForEachDeclStmt(DeclGroupPtrTy dg) {
|
|
|
|
DeclGroupRef DG = dg.getAsVal<DeclGroupRef>();
|
2013-04-09 04:52:24 +08:00
|
|
|
|
|
|
|
// If we don't have a declaration, or we have an invalid declaration,
|
|
|
|
// just return.
|
|
|
|
if (DG.isNull() || !DG.isSingleDecl())
|
|
|
|
return;
|
|
|
|
|
|
|
|
Decl *decl = DG.getSingleDecl();
|
|
|
|
if (!decl || decl->isInvalidDecl())
|
|
|
|
return;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2013-04-09 04:52:24 +08:00
|
|
|
// Only variable declarations are permitted.
|
|
|
|
VarDecl *var = dyn_cast<VarDecl>(decl);
|
|
|
|
if (!var) {
|
|
|
|
Diag(decl->getLocation(), diag::err_non_variable_decl_in_for);
|
|
|
|
decl->setInvalidDecl();
|
|
|
|
return;
|
|
|
|
}
|
2011-06-16 07:02:42 +08:00
|
|
|
|
2009-11-20 06:12:37 +08:00
|
|
|
// suppress any potential 'unused variable' warning.
|
2011-06-16 07:02:42 +08:00
|
|
|
var->setUsed();
|
|
|
|
|
2011-06-17 14:42:21 +08:00
|
|
|
// foreach variables are never actually initialized in the way that
|
|
|
|
// the parser came up with.
|
|
|
|
var->setInit(0);
|
|
|
|
|
|
|
|
// In ARC, we don't need to retain the iteration variable of a fast
|
|
|
|
// enumeration loop. Rather than actually trying to catch that
|
|
|
|
// during declaration processing, we remove the consequences here.
|
2012-03-11 15:00:24 +08:00
|
|
|
if (getLangOpts().ObjCAutoRefCount) {
|
2011-06-17 14:42:21 +08:00
|
|
|
QualType type = var->getType();
|
|
|
|
|
|
|
|
// Only do this if we inferred the lifetime. Inferred lifetime
|
|
|
|
// will show up as a local qualifier because explicit lifetime
|
|
|
|
// should have shown up as an AttributedType instead.
|
|
|
|
if (type.getLocalQualifiers().getObjCLifetime() == Qualifiers::OCL_Strong) {
|
|
|
|
// Add 'const' and mark the variable as pseudo-strong.
|
|
|
|
var->setType(type.withConst());
|
|
|
|
var->setARCPseudoStrong(true);
|
2011-06-16 07:02:42 +08:00
|
|
|
}
|
|
|
|
}
|
2009-11-20 06:12:37 +08:00
|
|
|
}
|
|
|
|
|
2011-08-17 17:34:37 +08:00
|
|
|
/// \brief Diagnose unused '==' and '!=' as likely typos for '=' or '|='.
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
///
|
|
|
|
/// Adding a cast to void (or other expression wrappers) will prevent the
|
|
|
|
/// warning from firing.
|
2011-08-17 17:34:37 +08:00
|
|
|
static bool DiagnoseUnusedComparison(Sema &S, const Expr *E) {
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
SourceLocation Loc;
|
2011-08-17 16:38:11 +08:00
|
|
|
bool IsNotEqual, CanAssign;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
|
|
|
|
if (const BinaryOperator *Op = dyn_cast<BinaryOperator>(E)) {
|
|
|
|
if (Op->getOpcode() != BO_EQ && Op->getOpcode() != BO_NE)
|
2011-08-17 17:34:37 +08:00
|
|
|
return false;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
|
|
|
|
Loc = Op->getOperatorLoc();
|
2011-08-17 16:38:11 +08:00
|
|
|
IsNotEqual = Op->getOpcode() == BO_NE;
|
|
|
|
CanAssign = Op->getLHS()->IgnoreParenImpCasts()->isLValue();
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
} else if (const CXXOperatorCallExpr *Op = dyn_cast<CXXOperatorCallExpr>(E)) {
|
|
|
|
if (Op->getOperator() != OO_EqualEqual &&
|
|
|
|
Op->getOperator() != OO_ExclaimEqual)
|
2011-08-17 17:34:37 +08:00
|
|
|
return false;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
|
|
|
|
Loc = Op->getOperatorLoc();
|
2011-08-17 16:38:11 +08:00
|
|
|
IsNotEqual = Op->getOperator() == OO_ExclaimEqual;
|
|
|
|
CanAssign = Op->getArg(0)->IgnoreParenImpCasts()->isLValue();
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
} else {
|
|
|
|
// Not a typo-prone comparison.
|
2011-08-17 17:34:37 +08:00
|
|
|
return false;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// Suppress warnings when the operator, suspicious as it may be, comes from
|
|
|
|
// a macro expansion.
|
2013-01-12 08:54:16 +08:00
|
|
|
if (S.SourceMgr.isMacroBodyExpansion(Loc))
|
2011-08-17 17:34:37 +08:00
|
|
|
return false;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
|
2011-08-17 17:34:37 +08:00
|
|
|
S.Diag(Loc, diag::warn_unused_comparison)
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
<< (unsigned)IsNotEqual << E->getSourceRange();
|
|
|
|
|
2011-08-17 16:38:11 +08:00
|
|
|
// If the LHS is a plausible entity to assign to, provide a fixit hint to
|
|
|
|
// correct common typos.
|
|
|
|
if (CanAssign) {
|
|
|
|
if (IsNotEqual)
|
|
|
|
S.Diag(Loc, diag::note_inequality_comparison_to_or_assign)
|
|
|
|
<< FixItHint::CreateReplacement(Loc, "|=");
|
|
|
|
else
|
|
|
|
S.Diag(Loc, diag::note_equality_comparison_to_assign)
|
|
|
|
<< FixItHint::CreateReplacement(Loc, "=");
|
|
|
|
}
|
2011-08-17 17:34:37 +08:00
|
|
|
|
|
|
|
return true;
|
Introduce a new warning, -Wtop-level-comparison. This warning is
a complement to the warnings we provide in condition expressions. Much
like we warn on conditions such as:
int x, y;
...
if (x = y) ... // Almost always a typo of '=='
This warning applies the complementary logic to "top-level" statements,
or statements whose value is not consumed or used in some way:
int x, y;
...
x == y; // Almost always a type for '='
We also mirror the '!=' vs. '|=' logic.
The warning is designed to fire even for overloaded operators for two reasons:
1) Especially in the presence of widespread templates that assume
operator== and operator!= perform the expected comparison operations,
it seems unreasonable to suppress warnings on the offchance that
a user has written a class that abuses these operators, embedding
side-effects or other magic within them.
2) There is a trivial source modification to silence the warning for
truly exceptional cases:
(void)(x == y); // No warning
A (greatly reduced) form of this warning has already caught a number of
bugs in our codebase, so there is precedent for it actually firing. That
said, its currently off by default, but enabled under -Wall.
There are several fixmes left here that I'm working on in follow-up
patches, including de-duplicating warnings from -Wunused, sharing code
with -Wunused's implementation (and creating a nice place to hook
diagnostics on "top-level" statements), and handling cases where a proxy
object with a bool conversion is returned, hiding the operation in the
cleanup AST nodes.
Suggestions for any of this code more than welcome. Also, I'd really
love suggestions for better naming than "top-level".
llvm-svn: 137819
2011-08-17 16:38:04 +08:00
|
|
|
}
|
|
|
|
|
2009-07-31 06:17:18 +08:00
|
|
|
void Sema::DiagnoseUnusedExprResult(const Stmt *S) {
|
2010-09-20 05:21:10 +08:00
|
|
|
if (const LabelStmt *Label = dyn_cast_or_null<LabelStmt>(S))
|
|
|
|
return DiagnoseUnusedExprResult(Label->getSubStmt());
|
|
|
|
|
2009-07-31 06:39:03 +08:00
|
|
|
const Expr *E = dyn_cast_or_null<Expr>(S);
|
2009-07-31 06:17:18 +08:00
|
|
|
if (!E)
|
|
|
|
return;
|
2013-01-17 10:06:08 +08:00
|
|
|
SourceLocation ExprLoc = E->IgnoreParens()->getExprLoc();
|
2013-02-27 03:34:08 +08:00
|
|
|
// In most cases, we don't want to warn if the expression is written in a
|
|
|
|
// macro body, or if the macro comes from a system header. If the offending
|
|
|
|
// expression is a call to a function with the warn_unused_result attribute,
|
|
|
|
// we warn no matter the location. Because of the order in which the various
|
|
|
|
// checks need to happen, we factor out the macro-related test here.
|
|
|
|
bool ShouldSuppress =
|
|
|
|
SourceMgr.isMacroBodyExpansion(ExprLoc) ||
|
|
|
|
SourceMgr.isInSystemMacro(ExprLoc);
|
2009-07-31 06:17:18 +08:00
|
|
|
|
2012-05-24 08:47:05 +08:00
|
|
|
const Expr *WarnExpr;
|
2009-07-31 06:17:18 +08:00
|
|
|
SourceLocation Loc;
|
|
|
|
SourceRange R1, R2;
|
2013-01-17 10:06:08 +08:00
|
|
|
if (!E->isUnusedResultAWarning(WarnExpr, Loc, R1, R2, Context))
|
2009-07-31 06:17:18 +08:00
|
|
|
return;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2012-09-01 06:39:21 +08:00
|
|
|
// If this is a GNU statement expression expanded from a macro, it is probably
|
|
|
|
// unused because it is a function-like macro that can be used as either an
|
|
|
|
// expression or statement. Don't warn, because it is almost certainly a
|
|
|
|
// false positive.
|
|
|
|
if (isa<StmtExpr>(E) && Loc.isMacroID())
|
|
|
|
return;
|
|
|
|
|
2009-08-17 00:57:27 +08:00
|
|
|
// Okay, we have an unused result. Depending on what the base expression is,
|
|
|
|
// we might want to make a more specific diagnostic. Check for one of these
|
|
|
|
// cases now.
|
|
|
|
unsigned DiagID = diag::warn_unused_expr;
|
2010-12-06 16:20:24 +08:00
|
|
|
if (const ExprWithCleanups *Temps = dyn_cast<ExprWithCleanups>(E))
|
2010-02-12 06:55:30 +08:00
|
|
|
E = Temps->getSubExpr();
|
2011-02-21 08:56:56 +08:00
|
|
|
if (const CXXBindTemporaryExpr *TempExpr = dyn_cast<CXXBindTemporaryExpr>(E))
|
|
|
|
E = TempExpr->getSubExpr();
|
2010-12-02 09:19:52 +08:00
|
|
|
|
2011-08-17 17:34:37 +08:00
|
|
|
if (DiagnoseUnusedComparison(*this, E))
|
|
|
|
return;
|
|
|
|
|
2012-05-24 08:47:05 +08:00
|
|
|
E = WarnExpr;
|
2009-10-13 12:53:48 +08:00
|
|
|
if (const CallExpr *CE = dyn_cast<CallExpr>(E)) {
|
2010-03-12 15:11:26 +08:00
|
|
|
if (E->getType()->isVoidType())
|
|
|
|
return;
|
|
|
|
|
2009-10-13 12:53:48 +08:00
|
|
|
// If the callee has attribute pure, const, or warn_unused_result, warn with
|
2013-02-27 03:34:08 +08:00
|
|
|
// a more specific message to make it clear what is happening. If the call
|
|
|
|
// is written in a macro body, only warn if it has the warn_unused_result
|
|
|
|
// attribute.
|
2009-12-21 07:11:08 +08:00
|
|
|
if (const Decl *FD = CE->getCalleeDecl()) {
|
2009-10-13 12:53:48 +08:00
|
|
|
if (FD->getAttr<WarnUnusedResultAttr>()) {
|
2011-08-05 07:11:04 +08:00
|
|
|
Diag(Loc, diag::warn_unused_result) << R1 << R2;
|
2009-10-13 12:53:48 +08:00
|
|
|
return;
|
|
|
|
}
|
2013-02-27 03:34:08 +08:00
|
|
|
if (ShouldSuppress)
|
|
|
|
return;
|
2009-10-13 12:53:48 +08:00
|
|
|
if (FD->getAttr<PureAttr>()) {
|
|
|
|
Diag(Loc, diag::warn_unused_call) << R1 << R2 << "pure";
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
if (FD->getAttr<ConstAttr>()) {
|
|
|
|
Diag(Loc, diag::warn_unused_call) << R1 << R2 << "const";
|
|
|
|
return;
|
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
}
|
2013-02-27 03:34:08 +08:00
|
|
|
} else if (ShouldSuppress)
|
|
|
|
return;
|
|
|
|
|
|
|
|
if (const ObjCMessageExpr *ME = dyn_cast<ObjCMessageExpr>(E)) {
|
2012-03-11 15:00:24 +08:00
|
|
|
if (getLangOpts().ObjCAutoRefCount && ME->isDelegateInitCall()) {
|
2011-06-16 07:02:42 +08:00
|
|
|
Diag(Loc, diag::err_arc_unused_init_message) << R1;
|
|
|
|
return;
|
|
|
|
}
|
2010-03-31 02:22:15 +08:00
|
|
|
const ObjCMethodDecl *MD = ME->getMethodDecl();
|
|
|
|
if (MD && MD->getAttr<WarnUnusedResultAttr>()) {
|
2011-08-05 07:11:04 +08:00
|
|
|
Diag(Loc, diag::warn_unused_result) << R1 << R2;
|
2010-03-31 02:22:15 +08:00
|
|
|
return;
|
|
|
|
}
|
2012-03-07 04:05:56 +08:00
|
|
|
} else if (const PseudoObjectExpr *POE = dyn_cast<PseudoObjectExpr>(E)) {
|
|
|
|
const Expr *Source = POE->getSyntacticForm();
|
|
|
|
if (isa<ObjCSubscriptRefExpr>(Source))
|
|
|
|
DiagID = diag::warn_unused_container_subscript_expr;
|
|
|
|
else
|
|
|
|
DiagID = diag::warn_unused_property_expr;
|
2010-04-17 06:09:46 +08:00
|
|
|
} else if (const CXXFunctionalCastExpr *FC
|
|
|
|
= dyn_cast<CXXFunctionalCastExpr>(E)) {
|
|
|
|
if (isa<CXXConstructExpr>(FC->getSubExpr()) ||
|
|
|
|
isa<CXXTemporaryObjectExpr>(FC->getSubExpr()))
|
|
|
|
return;
|
2010-03-31 02:22:15 +08:00
|
|
|
}
|
2010-04-07 06:24:14 +08:00
|
|
|
// Diagnose "(void*) blah" as a typo for "(void) blah".
|
|
|
|
else if (const CStyleCastExpr *CE = dyn_cast<CStyleCastExpr>(E)) {
|
|
|
|
TypeSourceInfo *TI = CE->getTypeInfoAsWritten();
|
|
|
|
QualType T = TI->getType();
|
|
|
|
|
|
|
|
// We really do want to use the non-canonical type here.
|
|
|
|
if (T == Context.VoidPtrTy) {
|
2013-02-19 06:06:02 +08:00
|
|
|
PointerTypeLoc TL = TI->getTypeLoc().castAs<PointerTypeLoc>();
|
2010-04-07 06:24:14 +08:00
|
|
|
|
|
|
|
Diag(Loc, diag::warn_unused_voidptr)
|
|
|
|
<< FixItHint::CreateRemoval(TL.getStarLoc());
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-05-24 08:47:05 +08:00
|
|
|
if (E->isGLValue() && E->getType().isVolatileQualified()) {
|
|
|
|
Diag(Loc, diag::warn_unused_volatile) << R1 << R2;
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2011-02-23 09:52:04 +08:00
|
|
|
DiagRuntimeBehavior(Loc, 0, PDiag(DiagID) << R1 << R2);
|
2009-07-31 06:17:18 +08:00
|
|
|
}
|
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
void Sema::ActOnStartOfCompoundStmt() {
|
|
|
|
PushCompoundScope();
|
|
|
|
}
|
|
|
|
|
|
|
|
void Sema::ActOnFinishOfCompoundStmt() {
|
|
|
|
PopCompoundScope();
|
|
|
|
}
|
|
|
|
|
|
|
|
sema::CompoundScopeInfo &Sema::getCurCompoundScope() const {
|
|
|
|
return getCurFunction()->CompoundScopes.back();
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2007-09-16 22:56:35 +08:00
|
|
|
Sema::ActOnCompoundStmt(SourceLocation L, SourceLocation R,
|
2008-12-21 20:04:03 +08:00
|
|
|
MultiStmtArg elts, bool isStmtExpr) {
|
|
|
|
unsigned NumElts = elts.size();
|
2012-08-24 07:38:35 +08:00
|
|
|
Stmt **Elts = elts.data();
|
2007-08-27 12:29:41 +08:00
|
|
|
// If we're in C89 mode, check that we don't have any decls after stmts. If
|
|
|
|
// so, emit an extension diagnostic.
|
2012-03-11 15:00:24 +08:00
|
|
|
if (!getLangOpts().C99 && !getLangOpts().CPlusPlus) {
|
2007-08-27 12:29:41 +08:00
|
|
|
// Note that __extension__ can be around a decl.
|
|
|
|
unsigned i = 0;
|
|
|
|
// Skip over all declarations.
|
|
|
|
for (; i != NumElts && isa<DeclStmt>(Elts[i]); ++i)
|
|
|
|
/*empty*/;
|
|
|
|
|
|
|
|
// We found the end of the list or a statement. Scan for another declstmt.
|
|
|
|
for (; i != NumElts && !isa<DeclStmt>(Elts[i]); ++i)
|
|
|
|
/*empty*/;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-27 12:29:41 +08:00
|
|
|
if (i != NumElts) {
|
2009-01-20 09:17:11 +08:00
|
|
|
Decl *D = *cast<DeclStmt>(Elts[i])->decl_begin();
|
2007-08-27 12:29:41 +08:00
|
|
|
Diag(D->getLocation(), diag::ext_mixed_decls_code);
|
|
|
|
}
|
|
|
|
}
|
2007-09-01 05:49:55 +08:00
|
|
|
// Warn about unused expressions in statements.
|
|
|
|
for (unsigned i = 0; i != NumElts; ++i) {
|
2009-07-31 06:17:18 +08:00
|
|
|
// Ignore statements that are last in a statement expression.
|
|
|
|
if (isStmtExpr && i == NumElts - 1)
|
2007-09-01 05:49:55 +08:00
|
|
|
continue;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-31 06:17:18 +08:00
|
|
|
DiagnoseUnusedExprResult(Elts[i]);
|
2007-09-01 05:49:55 +08:00
|
|
|
}
|
2008-12-21 20:04:03 +08:00
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
// Check for suspicious empty body (null statement) in `for' and `while'
|
|
|
|
// statements. Don't do anything for template instantiations, this just adds
|
|
|
|
// noise.
|
|
|
|
if (NumElts != 0 && !CurrentInstantiationScope &&
|
|
|
|
getCurCompoundScope().HasEmptyLoopBodies) {
|
|
|
|
for (unsigned i = 0; i != NumElts - 1; ++i)
|
|
|
|
DiagnoseEmptyLoopBody(Elts[i], Elts[i + 1]);
|
|
|
|
}
|
|
|
|
|
2012-12-30 04:03:39 +08:00
|
|
|
return Owned(new (Context) CompoundStmt(Context,
|
|
|
|
llvm::makeArrayRef(Elts, NumElts),
|
|
|
|
L, R));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnCaseStmt(SourceLocation CaseLoc, Expr *LHSVal,
|
|
|
|
SourceLocation DotDotDotLoc, Expr *RHSVal,
|
2009-03-04 12:23:07 +08:00
|
|
|
SourceLocation ColonLoc) {
|
2010-08-24 07:25:46 +08:00
|
|
|
assert((LHSVal != 0) && "missing expression in case statement");
|
2008-12-29 00:13:43 +08:00
|
|
|
|
2012-01-19 07:55:52 +08:00
|
|
|
if (getCurFunction()->SwitchStack.empty()) {
|
|
|
|
Diag(CaseLoc, diag::err_case_not_in_switch);
|
2009-03-04 12:23:07 +08:00
|
|
|
return StmtError();
|
2012-01-19 07:55:52 +08:00
|
|
|
}
|
2007-05-09 05:09:37 +08:00
|
|
|
|
2013-01-02 19:42:31 +08:00
|
|
|
if (!getLangOpts().CPlusPlus11) {
|
2012-01-19 07:55:52 +08:00
|
|
|
// C99 6.8.4.2p3: The expression shall be an integer constant.
|
|
|
|
// However, GCC allows any evaluatable integer expression.
|
2012-02-04 17:53:13 +08:00
|
|
|
if (!LHSVal->isTypeDependent() && !LHSVal->isValueDependent()) {
|
|
|
|
LHSVal = VerifyIntegerConstantExpression(LHSVal).take();
|
|
|
|
if (!LHSVal)
|
|
|
|
return StmtError();
|
|
|
|
}
|
2008-12-29 00:13:43 +08:00
|
|
|
|
2012-01-19 07:55:52 +08:00
|
|
|
// GCC extension: The expression shall be an integer constant.
|
2008-12-29 00:13:43 +08:00
|
|
|
|
2012-02-04 17:53:13 +08:00
|
|
|
if (RHSVal && !RHSVal->isTypeDependent() && !RHSVal->isValueDependent()) {
|
|
|
|
RHSVal = VerifyIntegerConstantExpression(RHSVal).take();
|
|
|
|
// Recover from an error by just forgetting about it.
|
2012-01-19 07:55:52 +08:00
|
|
|
}
|
2007-07-24 01:05:23 +08:00
|
|
|
}
|
2013-01-25 06:11:45 +08:00
|
|
|
|
|
|
|
LHSVal = ActOnFinishFullExpr(LHSVal, LHSVal->getExprLoc(), false,
|
|
|
|
getLangOpts().CPlusPlus11).take();
|
|
|
|
if (RHSVal)
|
|
|
|
RHSVal = ActOnFinishFullExpr(RHSVal, RHSVal->getExprLoc(), false,
|
|
|
|
getLangOpts().CPlusPlus11).take();
|
2007-06-03 09:44:43 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
CaseStmt *CS = new (Context) CaseStmt(LHSVal, RHSVal, CaseLoc, DotDotDotLoc,
|
|
|
|
ColonLoc);
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->SwitchStack.back()->addSwitchCase(CS);
|
2008-12-29 00:13:43 +08:00
|
|
|
return Owned(CS);
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2009-03-04 12:23:07 +08:00
|
|
|
/// ActOnCaseStmtBody - This installs a statement as the body of a case.
|
2010-08-24 07:25:46 +08:00
|
|
|
void Sema::ActOnCaseStmtBody(Stmt *caseStmt, Stmt *SubStmt) {
|
2011-08-18 10:04:29 +08:00
|
|
|
DiagnoseUnusedExprResult(SubStmt);
|
|
|
|
|
2009-03-04 12:23:07 +08:00
|
|
|
CaseStmt *CS = static_cast<CaseStmt*>(caseStmt);
|
|
|
|
CS->setSubStmt(SubStmt);
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2009-09-09 23:08:12 +08:00
|
|
|
Sema::ActOnDefaultStmt(SourceLocation DefaultLoc, SourceLocation ColonLoc,
|
2010-08-24 07:25:46 +08:00
|
|
|
Stmt *SubStmt, Scope *CurScope) {
|
2011-08-18 10:04:29 +08:00
|
|
|
DiagnoseUnusedExprResult(SubStmt);
|
|
|
|
|
2010-08-25 16:40:02 +08:00
|
|
|
if (getCurFunction()->SwitchStack.empty()) {
|
2007-07-21 11:00:26 +08:00
|
|
|
Diag(DefaultLoc, diag::err_default_not_in_switch);
|
2008-12-29 00:13:43 +08:00
|
|
|
return Owned(SubStmt);
|
2007-07-21 11:00:26 +08:00
|
|
|
}
|
2008-12-29 00:13:43 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
DefaultStmt *DS = new (Context) DefaultStmt(DefaultLoc, ColonLoc, SubStmt);
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->SwitchStack.back()->addSwitchCase(DS);
|
2008-12-29 00:13:43 +08:00
|
|
|
return Owned(DS);
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2011-02-18 04:34:02 +08:00
|
|
|
StmtResult
|
|
|
|
Sema::ActOnLabelStmt(SourceLocation IdentLoc, LabelDecl *TheDecl,
|
|
|
|
SourceLocation ColonLoc, Stmt *SubStmt) {
|
2011-02-17 15:39:24 +08:00
|
|
|
// If the label was multiply defined, reject it now.
|
|
|
|
if (TheDecl->getStmt()) {
|
|
|
|
Diag(IdentLoc, diag::err_redefinition_of_label) << TheDecl->getDeclName();
|
|
|
|
Diag(TheDecl->getLocation(), diag::note_previous_definition);
|
2009-01-11 08:38:46 +08:00
|
|
|
return Owned(SubStmt);
|
2007-05-28 14:28:18 +08:00
|
|
|
}
|
2009-01-11 08:38:46 +08:00
|
|
|
|
2011-02-17 15:39:24 +08:00
|
|
|
// Otherwise, things are good. Fill in the declaration and return it.
|
|
|
|
LabelStmt *LS = new (Context) LabelStmt(IdentLoc, TheDecl, SubStmt);
|
|
|
|
TheDecl->setStmt(LS);
|
2012-10-16 05:07:44 +08:00
|
|
|
if (!TheDecl->isGnuLocal()) {
|
|
|
|
TheDecl->setLocStart(IdentLoc);
|
2011-03-04 02:24:14 +08:00
|
|
|
TheDecl->setLocation(IdentLoc);
|
2012-10-16 05:07:44 +08:00
|
|
|
}
|
2011-02-17 15:39:24 +08:00
|
|
|
return Owned(LS);
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2012-04-14 08:33:13 +08:00
|
|
|
StmtResult Sema::ActOnAttributedStmt(SourceLocation AttrLoc,
|
2012-07-09 18:04:07 +08:00
|
|
|
ArrayRef<const Attr*> Attrs,
|
2012-04-14 08:33:13 +08:00
|
|
|
Stmt *SubStmt) {
|
2012-07-09 18:04:07 +08:00
|
|
|
// Fill in the declaration and return it.
|
|
|
|
AttributedStmt *LS = AttributedStmt::Create(Context, AttrLoc, Attrs, SubStmt);
|
2012-04-14 08:33:13 +08:00
|
|
|
return Owned(LS);
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-21 17:40:31 +08:00
|
|
|
Sema::ActOnIfStmt(SourceLocation IfLoc, FullExprArg CondVal, Decl *CondVar,
|
2010-11-20 10:04:01 +08:00
|
|
|
Stmt *thenStmt, SourceLocation ElseLoc,
|
|
|
|
Stmt *elseStmt) {
|
2013-02-16 02:34:13 +08:00
|
|
|
// If the condition was invalid, discard the if statement. We could recover
|
|
|
|
// better by replacing it with a valid expr, but don't do that yet.
|
|
|
|
if (!CondVal.get() && !CondVar) {
|
|
|
|
getCurFunction()->setHasDroppedStmt();
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
ExprResult CondResult(CondVal.release());
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-11-24 07:44:04 +08:00
|
|
|
VarDecl *ConditionVar = 0;
|
2010-08-21 17:40:31 +08:00
|
|
|
if (CondVar) {
|
|
|
|
ConditionVar = cast<VarDecl>(CondVar);
|
2010-05-07 01:25:47 +08:00
|
|
|
CondResult = CheckConditionVariable(ConditionVar, IfLoc, true);
|
2009-11-25 08:27:52 +08:00
|
|
|
if (CondResult.isInvalid())
|
|
|
|
return StmtError();
|
2009-11-24 07:44:04 +08:00
|
|
|
}
|
2009-11-25 08:27:52 +08:00
|
|
|
Expr *ConditionExpr = CondResult.takeAs<Expr>();
|
|
|
|
if (!ConditionExpr)
|
|
|
|
return StmtError();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2009-07-31 06:39:03 +08:00
|
|
|
DiagnoseUnusedExprResult(thenStmt);
|
2007-05-29 10:14:17 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
if (!elseStmt) {
|
2012-02-15 06:14:32 +08:00
|
|
|
DiagnoseEmptyStmtBody(ConditionExpr->getLocEnd(), thenStmt,
|
|
|
|
diag::warn_empty_if_body);
|
2007-10-11 04:50:11 +08:00
|
|
|
}
|
|
|
|
|
2009-07-31 06:39:03 +08:00
|
|
|
DiagnoseUnusedExprResult(elseStmt);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2011-01-27 15:10:08 +08:00
|
|
|
return Owned(new (Context) IfStmt(Context, IfLoc, ConditionVar, ConditionExpr,
|
2010-11-20 10:04:01 +08:00
|
|
|
thenStmt, ElseLoc, elseStmt));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
2007-05-29 10:14:17 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
/// ConvertIntegerToTypeWarnOnOverflow - Convert the specified APInt to have
|
|
|
|
/// the specified width and sign. If an overflow occurs, detect it and emit
|
|
|
|
/// the specified diagnostic.
|
|
|
|
void Sema::ConvertIntegerToTypeWarnOnOverflow(llvm::APSInt &Val,
|
|
|
|
unsigned NewWidth, bool NewSign,
|
2009-09-09 23:08:12 +08:00
|
|
|
SourceLocation Loc,
|
2007-08-23 13:46:52 +08:00
|
|
|
unsigned DiagID) {
|
|
|
|
// Perform a conversion to the promoted condition type if needed.
|
|
|
|
if (NewWidth > Val.getBitWidth()) {
|
|
|
|
// If this is an extension, just do it.
|
2010-12-07 16:25:34 +08:00
|
|
|
Val = Val.extend(NewWidth);
|
2007-08-23 13:46:52 +08:00
|
|
|
Val.setIsSigned(NewSign);
|
2010-03-01 09:04:55 +08:00
|
|
|
|
|
|
|
// If the input was signed and negative and the output is
|
|
|
|
// unsigned, don't bother to warn: this is implementation-defined
|
|
|
|
// behavior.
|
|
|
|
// FIXME: Introduce a second, default-ignored warning for this case?
|
2007-08-23 13:46:52 +08:00
|
|
|
} else if (NewWidth < Val.getBitWidth()) {
|
|
|
|
// If this is a truncation, check for overflow.
|
|
|
|
llvm::APSInt ConvVal(Val);
|
2010-12-07 16:25:34 +08:00
|
|
|
ConvVal = ConvVal.trunc(NewWidth);
|
2007-08-24 06:08:35 +08:00
|
|
|
ConvVal.setIsSigned(NewSign);
|
2010-12-07 16:25:34 +08:00
|
|
|
ConvVal = ConvVal.extend(Val.getBitWidth());
|
2007-08-24 06:08:35 +08:00
|
|
|
ConvVal.setIsSigned(Val.isSigned());
|
2007-08-23 13:46:52 +08:00
|
|
|
if (ConvVal != Val)
|
2008-11-20 14:06:08 +08:00
|
|
|
Diag(Loc, DiagID) << Val.toString(10) << ConvVal.toString(10);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
// Regardless of whether a diagnostic was emitted, really do the
|
|
|
|
// truncation.
|
2010-12-07 16:25:34 +08:00
|
|
|
Val = Val.trunc(NewWidth);
|
2007-08-24 06:08:35 +08:00
|
|
|
Val.setIsSigned(NewSign);
|
2007-08-23 13:46:52 +08:00
|
|
|
} else if (NewSign != Val.isSigned()) {
|
|
|
|
// Convert the sign to match the sign of the condition. This can cause
|
|
|
|
// overflow as well: unsigned(INTMIN)
|
2011-01-27 15:10:08 +08:00
|
|
|
// We don't diagnose this overflow, because it is implementation-defined
|
2010-02-18 08:56:01 +08:00
|
|
|
// behavior.
|
|
|
|
// FIXME: Introduce a second, default-ignored warning for this case?
|
2007-08-23 13:46:52 +08:00
|
|
|
llvm::APSInt OldVal(Val);
|
|
|
|
Val.setIsSigned(NewSign);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-08-24 02:29:20 +08:00
|
|
|
namespace {
|
|
|
|
struct CaseCompareFunctor {
|
|
|
|
bool operator()(const std::pair<llvm::APSInt, CaseStmt*> &LHS,
|
|
|
|
const llvm::APSInt &RHS) {
|
|
|
|
return LHS.first < RHS;
|
|
|
|
}
|
2007-09-04 02:31:57 +08:00
|
|
|
bool operator()(const std::pair<llvm::APSInt, CaseStmt*> &LHS,
|
|
|
|
const std::pair<llvm::APSInt, CaseStmt*> &RHS) {
|
|
|
|
return LHS.first < RHS.first;
|
|
|
|
}
|
2007-08-24 02:29:20 +08:00
|
|
|
bool operator()(const llvm::APSInt &LHS,
|
|
|
|
const std::pair<llvm::APSInt, CaseStmt*> &RHS) {
|
|
|
|
return LHS < RHS.first;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2007-09-22 02:15:22 +08:00
|
|
|
/// CmpCaseVals - Comparison predicate for sorting case values.
|
|
|
|
///
|
|
|
|
static bool CmpCaseVals(const std::pair<llvm::APSInt, CaseStmt*>& lhs,
|
|
|
|
const std::pair<llvm::APSInt, CaseStmt*>& rhs) {
|
|
|
|
if (lhs.first < rhs.first)
|
|
|
|
return true;
|
|
|
|
|
|
|
|
if (lhs.first == rhs.first &&
|
|
|
|
lhs.second->getCaseLoc().getRawEncoding()
|
|
|
|
< rhs.second->getCaseLoc().getRawEncoding())
|
|
|
|
return true;
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2010-02-09 06:24:16 +08:00
|
|
|
/// CmpEnumVals - Comparison predicate for sorting enumeration values.
|
|
|
|
///
|
|
|
|
static bool CmpEnumVals(const std::pair<llvm::APSInt, EnumConstantDecl*>& lhs,
|
|
|
|
const std::pair<llvm::APSInt, EnumConstantDecl*>& rhs)
|
|
|
|
{
|
|
|
|
return lhs.first < rhs.first;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// EqEnumVals - Comparison preficate for uniqing enumeration values.
|
|
|
|
///
|
|
|
|
static bool EqEnumVals(const std::pair<llvm::APSInt, EnumConstantDecl*>& lhs,
|
|
|
|
const std::pair<llvm::APSInt, EnumConstantDecl*>& rhs)
|
|
|
|
{
|
|
|
|
return lhs.first == rhs.first;
|
|
|
|
}
|
|
|
|
|
2009-10-17 00:45:22 +08:00
|
|
|
/// GetTypeBeforeIntegralPromotion - Returns the pre-promotion type of
|
|
|
|
/// potentially integral-promoted expression @p expr.
|
2011-08-06 15:30:58 +08:00
|
|
|
static QualType GetTypeBeforeIntegralPromotion(Expr *&expr) {
|
|
|
|
if (ExprWithCleanups *cleanups = dyn_cast<ExprWithCleanups>(expr))
|
|
|
|
expr = cleanups->getSubExpr();
|
|
|
|
while (ImplicitCastExpr *impcast = dyn_cast<ImplicitCastExpr>(expr)) {
|
|
|
|
if (impcast->getCastKind() != CK_IntegralCast) break;
|
|
|
|
expr = impcast->getSubExpr();
|
2009-10-17 00:45:22 +08:00
|
|
|
}
|
|
|
|
return expr->getType();
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2011-01-27 15:10:08 +08:00
|
|
|
Sema::ActOnStartOfSwitchStmt(SourceLocation SwitchLoc, Expr *Cond,
|
2010-08-21 17:40:31 +08:00
|
|
|
Decl *CondVar) {
|
2010-08-24 14:29:42 +08:00
|
|
|
ExprResult CondResult;
|
2010-08-24 07:25:46 +08:00
|
|
|
|
2010-05-07 01:25:47 +08:00
|
|
|
VarDecl *ConditionVar = 0;
|
2010-08-21 17:40:31 +08:00
|
|
|
if (CondVar) {
|
|
|
|
ConditionVar = cast<VarDecl>(CondVar);
|
2010-08-24 07:25:46 +08:00
|
|
|
CondResult = CheckConditionVariable(ConditionVar, SourceLocation(), false);
|
|
|
|
if (CondResult.isInvalid())
|
2010-05-07 01:25:47 +08:00
|
|
|
return StmtError();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
Cond = CondResult.release();
|
2010-05-07 01:25:47 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
if (!Cond)
|
2010-05-07 01:25:47 +08:00
|
|
|
return StmtError();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
class SwitchConvertDiagnoser : public ICEConvertDiagnoser {
|
|
|
|
Expr *Cond;
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
public:
|
|
|
|
SwitchConvertDiagnoser(Expr *Cond)
|
|
|
|
: ICEConvertDiagnoser(false, true), Cond(Cond) { }
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder diagnoseNotInt(Sema &S, SourceLocation Loc,
|
|
|
|
QualType T) {
|
|
|
|
return S.Diag(Loc, diag::err_typecheck_statement_requires_integer) << T;
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder diagnoseIncomplete(Sema &S, SourceLocation Loc,
|
|
|
|
QualType T) {
|
|
|
|
return S.Diag(Loc, diag::err_switch_incomplete_class_type)
|
|
|
|
<< T << Cond->getSourceRange();
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder diagnoseExplicitConv(Sema &S, SourceLocation Loc,
|
|
|
|
QualType T,
|
|
|
|
QualType ConvTy) {
|
|
|
|
return S.Diag(Loc, diag::err_switch_explicit_conversion) << T << ConvTy;
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder noteExplicitConv(Sema &S, CXXConversionDecl *Conv,
|
|
|
|
QualType ConvTy) {
|
|
|
|
return S.Diag(Conv->getLocation(), diag::note_switch_conversion)
|
|
|
|
<< ConvTy->isEnumeralType() << ConvTy;
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder diagnoseAmbiguous(Sema &S, SourceLocation Loc,
|
|
|
|
QualType T) {
|
|
|
|
return S.Diag(Loc, diag::err_switch_multiple_conversions) << T;
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder noteAmbiguous(Sema &S, CXXConversionDecl *Conv,
|
|
|
|
QualType ConvTy) {
|
|
|
|
return S.Diag(Conv->getLocation(), diag::note_switch_conversion)
|
|
|
|
<< ConvTy->isEnumeralType() << ConvTy;
|
|
|
|
}
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2012-05-05 06:38:52 +08:00
|
|
|
virtual DiagnosticBuilder diagnoseConversion(Sema &S, SourceLocation Loc,
|
|
|
|
QualType T,
|
|
|
|
QualType ConvTy) {
|
|
|
|
return DiagnosticBuilder::getEmpty();
|
|
|
|
}
|
|
|
|
} SwitchDiagnoser(Cond);
|
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
CondResult
|
2012-05-05 06:38:52 +08:00
|
|
|
= ConvertToIntegralOrEnumerationType(SwitchLoc, Cond, SwitchDiagnoser,
|
2012-02-04 15:07:42 +08:00
|
|
|
/*AllowScopedEnumerations*/ true);
|
2010-08-24 07:25:46 +08:00
|
|
|
if (CondResult.isInvalid()) return StmtError();
|
|
|
|
Cond = CondResult.take();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-08-06 15:30:58 +08:00
|
|
|
// C99 6.8.4.2p5 - Integer promotions are performed on the controlling expr.
|
|
|
|
CondResult = UsualUnaryConversions(Cond);
|
|
|
|
if (CondResult.isInvalid()) return StmtError();
|
|
|
|
Cond = CondResult.take();
|
|
|
|
|
2010-08-21 17:40:31 +08:00
|
|
|
if (!CondVar) {
|
2013-01-15 06:39:08 +08:00
|
|
|
CondResult = ActOnFinishFullExpr(Cond, SwitchLoc);
|
2010-08-24 07:25:46 +08:00
|
|
|
if (CondResult.isInvalid())
|
2010-05-07 01:25:47 +08:00
|
|
|
return StmtError();
|
2010-08-24 07:25:46 +08:00
|
|
|
Cond = CondResult.take();
|
2010-05-07 01:25:47 +08:00
|
|
|
}
|
2010-08-01 08:26:45 +08:00
|
|
|
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->setHasBranchIntoScope();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
SwitchStmt *SS = new (Context) SwitchStmt(Context, ConditionVar, Cond);
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->SwitchStack.push_back(SS);
|
2010-05-07 01:25:47 +08:00
|
|
|
return Owned(SS);
|
2010-01-24 09:50:29 +08:00
|
|
|
}
|
|
|
|
|
2010-10-02 06:05:14 +08:00
|
|
|
static void AdjustAPSInt(llvm::APSInt &Val, unsigned BitWidth, bool IsSigned) {
|
|
|
|
if (Val.getBitWidth() < BitWidth)
|
2010-12-07 16:25:34 +08:00
|
|
|
Val = Val.extend(BitWidth);
|
2010-10-02 06:05:14 +08:00
|
|
|
else if (Val.getBitWidth() > BitWidth)
|
2010-12-07 16:25:34 +08:00
|
|
|
Val = Val.trunc(BitWidth);
|
2010-10-02 06:05:14 +08:00
|
|
|
Val.setIsSigned(IsSigned);
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnFinishSwitchStmt(SourceLocation SwitchLoc, Stmt *Switch,
|
|
|
|
Stmt *BodyStmt) {
|
|
|
|
SwitchStmt *SS = cast<SwitchStmt>(Switch);
|
2010-08-25 16:40:02 +08:00
|
|
|
assert(SS == getCurFunction()->SwitchStack.back() &&
|
|
|
|
"switch stack missing push/pop!");
|
2009-01-11 08:38:46 +08:00
|
|
|
|
2007-09-02 05:08:38 +08:00
|
|
|
SS->setBody(BodyStmt, SwitchLoc);
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->SwitchStack.pop_back();
|
2007-07-22 15:07:56 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
Expr *CondExpr = SS->getCond();
|
2011-08-06 15:30:58 +08:00
|
|
|
if (!CondExpr) return StmtError();
|
2009-01-11 08:38:46 +08:00
|
|
|
|
2009-11-25 13:02:21 +08:00
|
|
|
QualType CondType = CondExpr->getType();
|
2011-08-06 15:30:58 +08:00
|
|
|
|
|
|
|
Expr *CondExprBeforePromotion = CondExpr;
|
|
|
|
QualType CondTypeBeforePromotion =
|
|
|
|
GetTypeBeforeIntegralPromotion(CondExprBeforePromotion);
|
2009-11-23 21:46:08 +08:00
|
|
|
|
2009-10-17 00:45:22 +08:00
|
|
|
// C++ 6.4.2.p2:
|
|
|
|
// Integral promotions are performed (on the switch condition).
|
|
|
|
//
|
|
|
|
// A case value unrepresentable by the original switch condition
|
|
|
|
// type (before the promotion) doesn't make sense, even when it can
|
|
|
|
// be represented by the promoted type. Therefore we need to find
|
|
|
|
// the pre-promotion type of the switch condition.
|
2009-10-18 03:32:54 +08:00
|
|
|
if (!CondExpr->isTypeDependent()) {
|
2010-06-30 07:25:20 +08:00
|
|
|
// We have already converted the expression to an integral or enumeration
|
2011-01-27 15:10:08 +08:00
|
|
|
// type, when we started the switch statement. If we don't have an
|
2010-06-30 07:25:20 +08:00
|
|
|
// appropriate type now, just return an error.
|
|
|
|
if (!CondType->isIntegralOrEnumerationType())
|
2009-10-18 03:32:54 +08:00
|
|
|
return StmtError();
|
|
|
|
|
2010-04-17 07:34:13 +08:00
|
|
|
if (CondExpr->isKnownToHaveBooleanValue()) {
|
2009-10-18 03:32:54 +08:00
|
|
|
// switch(bool_expr) {...} is often a programmer error, e.g.
|
|
|
|
// switch(n && mask) { ... } // Doh - should be "n & mask".
|
|
|
|
// One can always use an if statement instead of switch(bool_expr).
|
|
|
|
Diag(SwitchLoc, diag::warn_bool_switch_condition)
|
|
|
|
<< CondExpr->getSourceRange();
|
|
|
|
}
|
2007-07-22 15:07:56 +08:00
|
|
|
}
|
2009-01-11 08:38:46 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
// Get the bitwidth of the switched-on value before promotions. We must
|
|
|
|
// convert the integer case values to this width before comparison.
|
2009-09-09 23:08:12 +08:00
|
|
|
bool HasDependentValue
|
2009-05-16 07:57:33 +08:00
|
|
|
= CondExpr->isTypeDependent() || CondExpr->isValueDependent();
|
2009-09-09 23:08:12 +08:00
|
|
|
unsigned CondWidth
|
2011-02-24 15:31:28 +08:00
|
|
|
= HasDependentValue ? 0 : Context.getIntWidth(CondTypeBeforePromotion);
|
2012-08-11 01:56:09 +08:00
|
|
|
bool CondIsSigned
|
2011-05-21 00:38:50 +08:00
|
|
|
= CondTypeBeforePromotion->isSignedIntegerOrEnumerationType();
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
// Accumulate all of the case values in a vector so that we can sort them
|
|
|
|
// and detect duplicates. This vector contains the APInt for the case after
|
|
|
|
// it has been converted to the condition type.
|
2011-07-23 18:55:15 +08:00
|
|
|
typedef SmallVector<std::pair<llvm::APSInt, CaseStmt*>, 64> CaseValsTy;
|
2007-08-24 02:29:20 +08:00
|
|
|
CaseValsTy CaseVals;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
// Keep track of any GNU case ranges we see. The APSInt is the low value.
|
2010-02-09 06:24:16 +08:00
|
|
|
typedef std::vector<std::pair<llvm::APSInt, CaseStmt*> > CaseRangesTy;
|
|
|
|
CaseRangesTy CaseRanges;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
DefaultStmt *TheDefaultStmt = 0;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 14:23:56 +08:00
|
|
|
bool CaseListIsErroneous = false;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
for (SwitchCase *SC = SS->getSwitchCaseList(); SC && !HasDependentValue;
|
2007-07-22 15:07:56 +08:00
|
|
|
SC = SC->getNextSwitchCase()) {
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-07-22 15:07:56 +08:00
|
|
|
if (DefaultStmt *DS = dyn_cast<DefaultStmt>(SC)) {
|
2007-08-23 13:46:52 +08:00
|
|
|
if (TheDefaultStmt) {
|
|
|
|
Diag(DS->getDefaultLoc(), diag::err_multiple_default_labels_defined);
|
2008-11-24 07:12:31 +08:00
|
|
|
Diag(TheDefaultStmt->getDefaultLoc(), diag::note_duplicate_case_prev);
|
2009-01-11 08:38:46 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
// FIXME: Remove the default statement from the switch block so that
|
2009-05-16 15:39:55 +08:00
|
|
|
// we'll return a valid AST. This requires recursing down the AST and
|
|
|
|
// finding it, not something we are set up to do right now. For now,
|
|
|
|
// just lop the entire switch stmt out of the AST.
|
2007-08-23 14:23:56 +08:00
|
|
|
CaseListIsErroneous = true;
|
2007-07-22 15:07:56 +08:00
|
|
|
}
|
2007-08-23 13:46:52 +08:00
|
|
|
TheDefaultStmt = DS;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 13:46:52 +08:00
|
|
|
} else {
|
|
|
|
CaseStmt *CS = cast<CaseStmt>(SC);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2008-01-17 03:17:22 +08:00
|
|
|
Expr *Lo = CS->getLHS();
|
2009-05-16 07:57:33 +08:00
|
|
|
|
|
|
|
if (Lo->isTypeDependent() || Lo->isValueDependent()) {
|
|
|
|
HasDependentValue = true;
|
|
|
|
break;
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2012-01-19 07:55:52 +08:00
|
|
|
llvm::APSInt LoVal;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2013-01-02 19:42:31 +08:00
|
|
|
if (getLangOpts().CPlusPlus11) {
|
2012-01-19 07:55:52 +08:00
|
|
|
// C++11 [stmt.switch]p2: the constant-expression shall be a converted
|
|
|
|
// constant expression of the promoted type of the switch condition.
|
|
|
|
ExprResult ConvLo =
|
|
|
|
CheckConvertedConstantExpression(Lo, CondType, LoVal, CCEK_CaseValue);
|
|
|
|
if (ConvLo.isInvalid()) {
|
|
|
|
CaseListIsErroneous = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
Lo = ConvLo.take();
|
|
|
|
} else {
|
|
|
|
// We already verified that the expression has a i-c-e value (C99
|
|
|
|
// 6.8.4.2p3) - get that value now.
|
2013-01-25 06:11:45 +08:00
|
|
|
LoVal = Lo->EvaluateKnownConstInt(Context);
|
2012-01-19 07:55:52 +08:00
|
|
|
|
|
|
|
// If the LHS is not the same type as the condition, insert an implicit
|
|
|
|
// cast.
|
|
|
|
Lo = DefaultLvalueConversion(Lo).take();
|
|
|
|
Lo = ImpCastExprToType(Lo, CondType, CK_IntegralCast).take();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Convert the value to the same width/sign as the condition had prior to
|
|
|
|
// integral promotions.
|
|
|
|
//
|
|
|
|
// FIXME: This causes us to reject valid code:
|
|
|
|
// switch ((char)c) { case 256: case 0: return 0; }
|
|
|
|
// Here we claim there is a duplicated condition value, but there is not.
|
2007-08-23 13:46:52 +08:00
|
|
|
ConvertIntegerToTypeWarnOnOverflow(LoVal, CondWidth, CondIsSigned,
|
2010-10-02 06:05:14 +08:00
|
|
|
Lo->getLocStart(),
|
2007-08-23 13:46:52 +08:00
|
|
|
diag::warn_case_value_overflow);
|
2007-07-18 10:28:47 +08:00
|
|
|
|
2008-01-17 03:17:22 +08:00
|
|
|
CS->setLHS(Lo);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2007-08-23 14:23:56 +08:00
|
|
|
// If this is a case range, remember it in CaseRanges, otherwise CaseVals.
|
2009-05-16 07:57:33 +08:00
|
|
|
if (CS->getRHS()) {
|
2009-09-09 23:08:12 +08:00
|
|
|
if (CS->getRHS()->isTypeDependent() ||
|
2009-05-16 07:57:33 +08:00
|
|
|
CS->getRHS()->isValueDependent()) {
|
|
|
|
HasDependentValue = true;
|
|
|
|
break;
|
|
|
|
}
|
2007-08-23 13:46:52 +08:00
|
|
|
CaseRanges.push_back(std::make_pair(LoVal, CS));
|
2009-09-09 23:08:12 +08:00
|
|
|
} else
|
2007-08-23 14:23:56 +08:00
|
|
|
CaseVals.push_back(std::make_pair(LoVal, CS));
|
2007-08-23 13:46:52 +08:00
|
|
|
}
|
|
|
|
}
|
2007-08-23 14:23:56 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
if (!HasDependentValue) {
|
2010-05-18 11:19:21 +08:00
|
|
|
// If we don't have a default statement, check whether the
|
|
|
|
// condition is constant.
|
|
|
|
llvm::APSInt ConstantCondValue;
|
|
|
|
bool HasConstantCond = false;
|
|
|
|
if (!HasDependentValue && !TheDefaultStmt) {
|
2011-10-29 08:50:52 +08:00
|
|
|
HasConstantCond
|
2011-12-29 03:48:30 +08:00
|
|
|
= CondExprBeforePromotion->EvaluateAsInt(ConstantCondValue, Context,
|
|
|
|
Expr::SE_AllowSideEffects);
|
|
|
|
assert(!HasConstantCond ||
|
|
|
|
(ConstantCondValue.getBitWidth() == CondWidth &&
|
|
|
|
ConstantCondValue.isSigned() == CondIsSigned));
|
2010-05-18 11:19:21 +08:00
|
|
|
}
|
2011-12-29 03:48:30 +08:00
|
|
|
bool ShouldCheckConstantCond = HasConstantCond;
|
2010-05-18 11:19:21 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Sort all the scalar case values so we can easily detect duplicates.
|
|
|
|
std::stable_sort(CaseVals.begin(), CaseVals.end(), CmpCaseVals);
|
|
|
|
|
|
|
|
if (!CaseVals.empty()) {
|
2010-05-18 11:19:21 +08:00
|
|
|
for (unsigned i = 0, e = CaseVals.size(); i != e; ++i) {
|
|
|
|
if (ShouldCheckConstantCond &&
|
|
|
|
CaseVals[i].first == ConstantCondValue)
|
|
|
|
ShouldCheckConstantCond = false;
|
|
|
|
|
|
|
|
if (i != 0 && CaseVals[i].first == CaseVals[i-1].first) {
|
2009-05-16 07:57:33 +08:00
|
|
|
// If we have a duplicate, report it.
|
2012-05-16 13:32:58 +08:00
|
|
|
// First, determine if either case value has a name
|
|
|
|
StringRef PrevString, CurrString;
|
|
|
|
Expr *PrevCase = CaseVals[i-1].second->getLHS()->IgnoreParenCasts();
|
|
|
|
Expr *CurrCase = CaseVals[i].second->getLHS()->IgnoreParenCasts();
|
|
|
|
if (DeclRefExpr *DeclRef = dyn_cast<DeclRefExpr>(PrevCase)) {
|
|
|
|
PrevString = DeclRef->getDecl()->getName();
|
|
|
|
}
|
|
|
|
if (DeclRefExpr *DeclRef = dyn_cast<DeclRefExpr>(CurrCase)) {
|
|
|
|
CurrString = DeclRef->getDecl()->getName();
|
|
|
|
}
|
2013-01-13 03:30:44 +08:00
|
|
|
SmallString<16> CaseValStr;
|
2012-05-17 00:11:17 +08:00
|
|
|
CaseVals[i-1].first.toString(CaseValStr);
|
2012-05-16 13:32:58 +08:00
|
|
|
|
|
|
|
if (PrevString == CurrString)
|
|
|
|
Diag(CaseVals[i].second->getLHS()->getLocStart(),
|
|
|
|
diag::err_duplicate_case) <<
|
2012-05-17 00:11:17 +08:00
|
|
|
(PrevString.empty() ? CaseValStr.str() : PrevString);
|
2012-05-16 13:32:58 +08:00
|
|
|
else
|
|
|
|
Diag(CaseVals[i].second->getLHS()->getLocStart(),
|
|
|
|
diag::err_duplicate_case_differing_expr) <<
|
2012-05-17 00:11:17 +08:00
|
|
|
(PrevString.empty() ? CaseValStr.str() : PrevString) <<
|
|
|
|
(CurrString.empty() ? CaseValStr.str() : CurrString) <<
|
2012-05-16 13:32:58 +08:00
|
|
|
CaseValStr;
|
|
|
|
|
2010-05-18 11:19:21 +08:00
|
|
|
Diag(CaseVals[i-1].second->getLHS()->getLocStart(),
|
2009-05-16 07:57:33 +08:00
|
|
|
diag::note_duplicate_case_prev);
|
2009-05-16 15:39:55 +08:00
|
|
|
// FIXME: We really want to remove the bogus case stmt from the
|
|
|
|
// substmt, but we have no way to do this right now.
|
2009-05-16 07:57:33 +08:00
|
|
|
CaseListIsErroneous = true;
|
|
|
|
}
|
2007-08-24 01:48:14 +08:00
|
|
|
}
|
2007-08-23 14:23:56 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Detect duplicate case ranges, which usually don't exist at all in
|
|
|
|
// the first place.
|
|
|
|
if (!CaseRanges.empty()) {
|
|
|
|
// Sort all the case ranges by their low value so we can easily detect
|
|
|
|
// overlaps between ranges.
|
|
|
|
std::stable_sort(CaseRanges.begin(), CaseRanges.end());
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Scan the ranges, computing the high values and removing empty ranges.
|
|
|
|
std::vector<llvm::APSInt> HiVals;
|
|
|
|
for (unsigned i = 0, e = CaseRanges.size(); i != e; ++i) {
|
2010-05-18 11:19:21 +08:00
|
|
|
llvm::APSInt &LoVal = CaseRanges[i].first;
|
2009-05-16 07:57:33 +08:00
|
|
|
CaseStmt *CR = CaseRanges[i].second;
|
|
|
|
Expr *Hi = CR->getRHS();
|
2012-01-19 07:55:52 +08:00
|
|
|
llvm::APSInt HiVal;
|
|
|
|
|
2013-01-02 19:42:31 +08:00
|
|
|
if (getLangOpts().CPlusPlus11) {
|
2012-01-19 07:55:52 +08:00
|
|
|
// C++11 [stmt.switch]p2: the constant-expression shall be a converted
|
|
|
|
// constant expression of the promoted type of the switch condition.
|
|
|
|
ExprResult ConvHi =
|
|
|
|
CheckConvertedConstantExpression(Hi, CondType, HiVal,
|
|
|
|
CCEK_CaseValue);
|
|
|
|
if (ConvHi.isInvalid()) {
|
|
|
|
CaseListIsErroneous = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
Hi = ConvHi.take();
|
|
|
|
} else {
|
|
|
|
HiVal = Hi->EvaluateKnownConstInt(Context);
|
|
|
|
|
|
|
|
// If the RHS is not the same type as the condition, insert an
|
|
|
|
// implicit cast.
|
|
|
|
Hi = DefaultLvalueConversion(Hi).take();
|
|
|
|
Hi = ImpCastExprToType(Hi, CondType, CK_IntegralCast).take();
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Convert the value to the same width/sign as the condition.
|
|
|
|
ConvertIntegerToTypeWarnOnOverflow(HiVal, CondWidth, CondIsSigned,
|
2010-10-02 06:05:14 +08:00
|
|
|
Hi->getLocStart(),
|
2009-05-16 07:57:33 +08:00
|
|
|
diag::warn_case_value_overflow);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
CR->setRHS(Hi);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// If the low value is bigger than the high value, the case is empty.
|
2010-05-18 11:19:21 +08:00
|
|
|
if (LoVal > HiVal) {
|
2009-05-16 07:57:33 +08:00
|
|
|
Diag(CR->getLHS()->getLocStart(), diag::warn_case_empty_range)
|
|
|
|
<< SourceRange(CR->getLHS()->getLocStart(),
|
2010-10-02 06:05:14 +08:00
|
|
|
Hi->getLocEnd());
|
2009-05-16 07:57:33 +08:00
|
|
|
CaseRanges.erase(CaseRanges.begin()+i);
|
|
|
|
--i, --e;
|
|
|
|
continue;
|
|
|
|
}
|
2010-05-18 11:19:21 +08:00
|
|
|
|
|
|
|
if (ShouldCheckConstantCond &&
|
|
|
|
LoVal <= ConstantCondValue &&
|
|
|
|
ConstantCondValue <= HiVal)
|
|
|
|
ShouldCheckConstantCond = false;
|
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
HiVals.push_back(HiVal);
|
2007-08-24 02:29:20 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Rescan the ranges, looking for overlap with singleton values and other
|
|
|
|
// ranges. Since the range list is sorted, we only need to compare case
|
|
|
|
// ranges with their neighbors.
|
|
|
|
for (unsigned i = 0, e = CaseRanges.size(); i != e; ++i) {
|
|
|
|
llvm::APSInt &CRLo = CaseRanges[i].first;
|
|
|
|
llvm::APSInt &CRHi = HiVals[i];
|
|
|
|
CaseStmt *CR = CaseRanges[i].second;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Check to see whether the case range overlaps with any
|
|
|
|
// singleton cases.
|
|
|
|
CaseStmt *OverlapStmt = 0;
|
|
|
|
llvm::APSInt OverlapVal(32);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Find the smallest value >= the lower bound. If I is in the
|
|
|
|
// case range, then we have overlap.
|
|
|
|
CaseValsTy::iterator I = std::lower_bound(CaseVals.begin(),
|
|
|
|
CaseVals.end(), CRLo,
|
|
|
|
CaseCompareFunctor());
|
|
|
|
if (I != CaseVals.end() && I->first < CRHi) {
|
|
|
|
OverlapVal = I->first; // Found overlap with scalar.
|
|
|
|
OverlapStmt = I->second;
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Find the smallest value bigger than the upper bound.
|
|
|
|
I = std::upper_bound(I, CaseVals.end(), CRHi, CaseCompareFunctor());
|
|
|
|
if (I != CaseVals.begin() && (I-1)->first >= CRLo) {
|
|
|
|
OverlapVal = (I-1)->first; // Found overlap with scalar.
|
|
|
|
OverlapStmt = (I-1)->second;
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
// Check to see if this case stmt overlaps with the subsequent
|
|
|
|
// case range.
|
|
|
|
if (i && CRLo <= HiVals[i-1]) {
|
|
|
|
OverlapVal = HiVals[i-1]; // Found overlap with range.
|
|
|
|
OverlapStmt = CaseRanges[i-1].second;
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-05-16 07:57:33 +08:00
|
|
|
if (OverlapStmt) {
|
|
|
|
// If we have a duplicate, report it.
|
|
|
|
Diag(CR->getLHS()->getLocStart(), diag::err_duplicate_case)
|
|
|
|
<< OverlapVal.toString(10);
|
2009-09-09 23:08:12 +08:00
|
|
|
Diag(OverlapStmt->getLHS()->getLocStart(),
|
2009-05-16 07:57:33 +08:00
|
|
|
diag::note_duplicate_case_prev);
|
2009-05-16 15:39:55 +08:00
|
|
|
// FIXME: We really want to remove the bogus case stmt from the
|
|
|
|
// substmt, but we have no way to do this right now.
|
2009-05-16 07:57:33 +08:00
|
|
|
CaseListIsErroneous = true;
|
|
|
|
}
|
2007-08-24 02:29:20 +08:00
|
|
|
}
|
2007-08-23 14:23:56 +08:00
|
|
|
}
|
2010-02-09 06:24:16 +08:00
|
|
|
|
2010-05-18 11:19:21 +08:00
|
|
|
// Complain if we have a constant condition and we didn't find a match.
|
|
|
|
if (!CaseListIsErroneous && ShouldCheckConstantCond) {
|
|
|
|
// TODO: it would be nice if we printed enums as enums, chars as
|
|
|
|
// chars, etc.
|
|
|
|
Diag(CondExpr->getExprLoc(), diag::warn_missing_case_for_condition)
|
|
|
|
<< ConstantCondValue.toString(10)
|
|
|
|
<< CondExpr->getSourceRange();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Check to see if switch is over an Enum and handles all of its
|
2010-09-09 08:05:53 +08:00
|
|
|
// values. We only issue a warning if there is not 'default:', but
|
|
|
|
// we still do the analysis to preserve this information in the AST
|
|
|
|
// (which can be used by flow-based analyes).
|
2010-05-18 11:19:21 +08:00
|
|
|
//
|
2010-09-17 01:09:42 +08:00
|
|
|
const EnumType *ET = CondTypeBeforePromotion->getAs<EnumType>();
|
2010-09-09 08:05:53 +08:00
|
|
|
|
2010-02-09 06:24:16 +08:00
|
|
|
// If switch has default case, then ignore it.
|
2010-09-09 08:05:53 +08:00
|
|
|
if (!CaseListIsErroneous && !HasConstantCond && ET) {
|
2010-02-09 06:24:16 +08:00
|
|
|
const EnumDecl *ED = ET->getDecl();
|
2011-07-23 18:55:15 +08:00
|
|
|
typedef SmallVector<std::pair<llvm::APSInt, EnumConstantDecl*>, 64>
|
2011-06-02 08:47:27 +08:00
|
|
|
EnumValsTy;
|
2010-02-09 06:24:16 +08:00
|
|
|
EnumValsTy EnumVals;
|
|
|
|
|
2010-05-18 11:19:21 +08:00
|
|
|
// Gather all enum values, set their type and sort them,
|
|
|
|
// allowing easier comparison with CaseVals.
|
|
|
|
for (EnumDecl::enumerator_iterator EDI = ED->enumerator_begin();
|
2010-10-02 06:05:14 +08:00
|
|
|
EDI != ED->enumerator_end(); ++EDI) {
|
|
|
|
llvm::APSInt Val = EDI->getInitVal();
|
|
|
|
AdjustAPSInt(Val, CondWidth, CondIsSigned);
|
2012-06-07 04:45:41 +08:00
|
|
|
EnumVals.push_back(std::make_pair(Val, *EDI));
|
2010-02-09 06:24:16 +08:00
|
|
|
}
|
|
|
|
std::stable_sort(EnumVals.begin(), EnumVals.end(), CmpEnumVals);
|
2010-05-18 11:19:21 +08:00
|
|
|
EnumValsTy::iterator EIend =
|
|
|
|
std::unique(EnumVals.begin(), EnumVals.end(), EqEnumVals);
|
2010-09-09 08:05:53 +08:00
|
|
|
|
|
|
|
// See which case values aren't in enum.
|
2012-01-22 10:31:55 +08:00
|
|
|
EnumValsTy::const_iterator EI = EnumVals.begin();
|
|
|
|
for (CaseValsTy::const_iterator CI = CaseVals.begin();
|
|
|
|
CI != CaseVals.end(); CI++) {
|
|
|
|
while (EI != EIend && EI->first < CI->first)
|
|
|
|
EI++;
|
|
|
|
if (EI == EIend || EI->first > CI->first)
|
|
|
|
Diag(CI->second->getLHS()->getExprLoc(), diag::warn_not_in_enum)
|
2012-03-22 04:56:29 +08:00
|
|
|
<< CondTypeBeforePromotion;
|
2012-01-22 10:31:55 +08:00
|
|
|
}
|
|
|
|
// See which of case ranges aren't in enum
|
|
|
|
EI = EnumVals.begin();
|
|
|
|
for (CaseRangesTy::const_iterator RI = CaseRanges.begin();
|
|
|
|
RI != CaseRanges.end() && EI != EIend; RI++) {
|
|
|
|
while (EI != EIend && EI->first < RI->first)
|
|
|
|
EI++;
|
|
|
|
|
|
|
|
if (EI == EIend || EI->first != RI->first) {
|
|
|
|
Diag(RI->second->getLHS()->getExprLoc(), diag::warn_not_in_enum)
|
2012-03-22 04:56:29 +08:00
|
|
|
<< CondTypeBeforePromotion;
|
2010-09-09 14:53:59 +08:00
|
|
|
}
|
2010-09-09 10:57:51 +08:00
|
|
|
|
2012-08-11 01:56:09 +08:00
|
|
|
llvm::APSInt Hi =
|
2012-01-22 10:31:55 +08:00
|
|
|
RI->second->getRHS()->EvaluateKnownConstInt(Context);
|
|
|
|
AdjustAPSInt(Hi, CondWidth, CondIsSigned);
|
|
|
|
while (EI != EIend && EI->first < Hi)
|
|
|
|
EI++;
|
|
|
|
if (EI == EIend || EI->first != Hi)
|
|
|
|
Diag(RI->second->getRHS()->getExprLoc(), diag::warn_not_in_enum)
|
2012-03-22 04:56:29 +08:00
|
|
|
<< CondTypeBeforePromotion;
|
2010-02-09 06:24:16 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-09-09 08:05:53 +08:00
|
|
|
// Check which enum vals aren't in switch
|
2010-02-09 06:24:16 +08:00
|
|
|
CaseValsTy::const_iterator CI = CaseVals.begin();
|
|
|
|
CaseRangesTy::const_iterator RI = CaseRanges.begin();
|
2010-09-09 08:05:53 +08:00
|
|
|
bool hasCasesNotInSwitch = false;
|
|
|
|
|
2011-07-23 18:55:15 +08:00
|
|
|
SmallVector<DeclarationName,8> UnhandledNames;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2012-01-22 10:31:55 +08:00
|
|
|
for (EI = EnumVals.begin(); EI != EIend; EI++){
|
2010-09-17 01:09:42 +08:00
|
|
|
// Drop unneeded case values
|
2010-02-09 06:24:16 +08:00
|
|
|
llvm::APSInt CIVal;
|
|
|
|
while (CI != CaseVals.end() && CI->first < EI->first)
|
|
|
|
CI++;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-02-09 06:24:16 +08:00
|
|
|
if (CI != CaseVals.end() && CI->first == EI->first)
|
|
|
|
continue;
|
|
|
|
|
2010-09-09 08:05:53 +08:00
|
|
|
// Drop unneeded case ranges
|
2010-02-09 06:24:16 +08:00
|
|
|
for (; RI != CaseRanges.end(); RI++) {
|
2011-10-11 02:28:20 +08:00
|
|
|
llvm::APSInt Hi =
|
|
|
|
RI->second->getRHS()->EvaluateKnownConstInt(Context);
|
2010-10-02 06:05:14 +08:00
|
|
|
AdjustAPSInt(Hi, CondWidth, CondIsSigned);
|
2010-02-09 06:24:16 +08:00
|
|
|
if (EI->first <= Hi)
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2010-09-09 08:05:53 +08:00
|
|
|
if (RI == CaseRanges.end() || EI->first < RI->first) {
|
2010-09-09 14:53:59 +08:00
|
|
|
hasCasesNotInSwitch = true;
|
2012-01-22 02:12:07 +08:00
|
|
|
UnhandledNames.push_back(EI->second->getDeclName());
|
2010-09-09 14:53:59 +08:00
|
|
|
}
|
2010-02-09 06:24:16 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2012-01-23 12:46:12 +08:00
|
|
|
if (TheDefaultStmt && UnhandledNames.empty())
|
|
|
|
Diag(TheDefaultStmt->getDefaultLoc(), diag::warn_unreachable_default);
|
2012-01-22 02:12:07 +08:00
|
|
|
|
2010-09-17 01:09:42 +08:00
|
|
|
// Produce a nice diagnostic if multiple values aren't handled.
|
|
|
|
switch (UnhandledNames.size()) {
|
|
|
|
case 0: break;
|
|
|
|
case 1:
|
2012-08-11 01:56:09 +08:00
|
|
|
Diag(CondExpr->getExprLoc(), TheDefaultStmt
|
2012-01-23 12:46:12 +08:00
|
|
|
? diag::warn_def_missing_case1 : diag::warn_missing_case1)
|
2010-09-17 01:09:42 +08:00
|
|
|
<< UnhandledNames[0];
|
|
|
|
break;
|
|
|
|
case 2:
|
2012-08-11 01:56:09 +08:00
|
|
|
Diag(CondExpr->getExprLoc(), TheDefaultStmt
|
2012-01-23 12:46:12 +08:00
|
|
|
? diag::warn_def_missing_case2 : diag::warn_missing_case2)
|
2010-09-17 01:09:42 +08:00
|
|
|
<< UnhandledNames[0] << UnhandledNames[1];
|
|
|
|
break;
|
|
|
|
case 3:
|
2012-01-23 12:46:12 +08:00
|
|
|
Diag(CondExpr->getExprLoc(), TheDefaultStmt
|
|
|
|
? diag::warn_def_missing_case3 : diag::warn_missing_case3)
|
2010-09-17 01:09:42 +08:00
|
|
|
<< UnhandledNames[0] << UnhandledNames[1] << UnhandledNames[2];
|
|
|
|
break;
|
|
|
|
default:
|
2012-01-23 12:46:12 +08:00
|
|
|
Diag(CondExpr->getExprLoc(), TheDefaultStmt
|
|
|
|
? diag::warn_def_missing_cases : diag::warn_missing_cases)
|
2010-09-17 01:09:42 +08:00
|
|
|
<< (unsigned)UnhandledNames.size()
|
|
|
|
<< UnhandledNames[0] << UnhandledNames[1] << UnhandledNames[2];
|
|
|
|
break;
|
|
|
|
}
|
2010-09-09 08:05:53 +08:00
|
|
|
|
|
|
|
if (!hasCasesNotInSwitch)
|
2010-09-09 14:53:59 +08:00
|
|
|
SS->setAllEnumCasesCovered();
|
2010-02-09 06:24:16 +08:00
|
|
|
}
|
2007-08-23 14:23:56 +08:00
|
|
|
}
|
2009-05-16 07:57:33 +08:00
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
DiagnoseEmptyStmtBody(CondExpr->getLocEnd(), BodyStmt,
|
|
|
|
diag::warn_empty_switch_body);
|
|
|
|
|
2009-05-16 15:39:55 +08:00
|
|
|
// FIXME: If the case list was broken is some way, we don't have a good system
|
|
|
|
// to patch it up. Instead, just return the whole substmt as broken.
|
2007-08-23 14:23:56 +08:00
|
|
|
if (CaseListIsErroneous)
|
2009-01-11 08:38:46 +08:00
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
return Owned(SS);
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2012-07-18 02:00:08 +08:00
|
|
|
void
|
|
|
|
Sema::DiagnoseAssignmentEnum(QualType DstType, QualType SrcType,
|
|
|
|
Expr *SrcExpr) {
|
|
|
|
unsigned DIAG = diag::warn_not_in_enum_assignement;
|
2012-08-11 01:56:09 +08:00
|
|
|
if (Diags.getDiagnosticLevel(DIAG, SrcExpr->getExprLoc())
|
2012-07-18 02:00:08 +08:00
|
|
|
== DiagnosticsEngine::Ignored)
|
|
|
|
return;
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-07-18 02:00:08 +08:00
|
|
|
if (const EnumType *ET = DstType->getAs<EnumType>())
|
|
|
|
if (!Context.hasSameType(SrcType, DstType) &&
|
|
|
|
SrcType->isIntegerType()) {
|
|
|
|
if (!SrcExpr->isTypeDependent() && !SrcExpr->isValueDependent() &&
|
|
|
|
SrcExpr->isIntegerConstantExpr(Context)) {
|
|
|
|
// Get the bitwidth of the enum value before promotions.
|
|
|
|
unsigned DstWith = Context.getIntWidth(DstType);
|
|
|
|
bool DstIsSigned = DstType->isSignedIntegerOrEnumerationType();
|
|
|
|
|
|
|
|
llvm::APSInt RhsVal = SrcExpr->EvaluateKnownConstInt(Context);
|
|
|
|
const EnumDecl *ED = ET->getDecl();
|
|
|
|
typedef SmallVector<std::pair<llvm::APSInt, EnumConstantDecl*>, 64>
|
|
|
|
EnumValsTy;
|
|
|
|
EnumValsTy EnumVals;
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-07-18 02:00:08 +08:00
|
|
|
// Gather all enum values, set their type and sort them,
|
|
|
|
// allowing easier comparison with rhs constant.
|
|
|
|
for (EnumDecl::enumerator_iterator EDI = ED->enumerator_begin();
|
|
|
|
EDI != ED->enumerator_end(); ++EDI) {
|
|
|
|
llvm::APSInt Val = EDI->getInitVal();
|
|
|
|
AdjustAPSInt(Val, DstWith, DstIsSigned);
|
|
|
|
EnumVals.push_back(std::make_pair(Val, *EDI));
|
|
|
|
}
|
|
|
|
if (EnumVals.empty())
|
|
|
|
return;
|
|
|
|
std::stable_sort(EnumVals.begin(), EnumVals.end(), CmpEnumVals);
|
|
|
|
EnumValsTy::iterator EIend =
|
|
|
|
std::unique(EnumVals.begin(), EnumVals.end(), EqEnumVals);
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-07-18 02:00:08 +08:00
|
|
|
// See which case values aren't in enum.
|
|
|
|
EnumValsTy::const_iterator EI = EnumVals.begin();
|
|
|
|
while (EI != EIend && EI->first < RhsVal)
|
|
|
|
EI++;
|
|
|
|
if (EI == EIend || EI->first != RhsVal) {
|
|
|
|
Diag(SrcExpr->getExprLoc(), diag::warn_not_in_enum_assignement)
|
|
|
|
<< DstType;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2011-01-27 15:10:08 +08:00
|
|
|
Sema::ActOnWhileStmt(SourceLocation WhileLoc, FullExprArg Cond,
|
2010-08-24 07:25:46 +08:00
|
|
|
Decl *CondVar, Stmt *Body) {
|
2010-08-24 14:29:42 +08:00
|
|
|
ExprResult CondResult(Cond.release());
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2009-11-25 05:15:44 +08:00
|
|
|
VarDecl *ConditionVar = 0;
|
2010-08-21 17:40:31 +08:00
|
|
|
if (CondVar) {
|
|
|
|
ConditionVar = cast<VarDecl>(CondVar);
|
2010-05-07 01:25:47 +08:00
|
|
|
CondResult = CheckConditionVariable(ConditionVar, WhileLoc, true);
|
2009-11-25 08:27:52 +08:00
|
|
|
if (CondResult.isInvalid())
|
|
|
|
return StmtError();
|
2009-11-25 05:15:44 +08:00
|
|
|
}
|
2010-08-24 07:25:46 +08:00
|
|
|
Expr *ConditionExpr = CondResult.take();
|
2009-11-25 08:27:52 +08:00
|
|
|
if (!ConditionExpr)
|
|
|
|
return StmtError();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
DiagnoseUnusedExprResult(Body);
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
if (isa<NullStmt>(Body))
|
|
|
|
getCurCompoundScope().setHasEmptyLoopBodies();
|
|
|
|
|
2010-06-22 07:44:13 +08:00
|
|
|
return Owned(new (Context) WhileStmt(Context, ConditionVar, ConditionExpr,
|
2010-08-24 07:25:46 +08:00
|
|
|
Body, WhileLoc));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnDoStmt(SourceLocation DoLoc, Stmt *Body,
|
2009-06-13 07:04:47 +08:00
|
|
|
SourceLocation WhileLoc, SourceLocation CondLParen,
|
2010-08-24 07:25:46 +08:00
|
|
|
Expr *Cond, SourceLocation CondRParen) {
|
|
|
|
assert(Cond && "ActOnDoStmt(): missing expression");
|
2009-01-17 07:28:06 +08:00
|
|
|
|
2011-04-09 02:41:53 +08:00
|
|
|
ExprResult CondResult = CheckBooleanCondition(Cond, DoLoc);
|
2012-11-19 06:28:42 +08:00
|
|
|
if (CondResult.isInvalid())
|
2009-10-13 05:59:07 +08:00
|
|
|
return StmtError();
|
2011-04-09 02:41:53 +08:00
|
|
|
Cond = CondResult.take();
|
2007-05-29 10:14:17 +08:00
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
CondResult = ActOnFinishFullExpr(Cond, DoLoc);
|
2010-08-24 07:25:46 +08:00
|
|
|
if (CondResult.isInvalid())
|
2010-05-07 01:25:47 +08:00
|
|
|
return StmtError();
|
2010-08-24 07:25:46 +08:00
|
|
|
Cond = CondResult.take();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
DiagnoseUnusedExprResult(Body);
|
2009-07-31 06:39:03 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(new (Context) DoStmt(Body, Cond, DoLoc, WhileLoc, CondRParen));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2012-05-01 02:01:30 +08:00
|
|
|
namespace {
|
|
|
|
// This visitor will traverse a conditional statement and store all
|
|
|
|
// the evaluated decls into a vector. Simple is set to true if none
|
|
|
|
// of the excluded constructs are used.
|
|
|
|
class DeclExtractor : public EvaluatedExprVisitor<DeclExtractor> {
|
|
|
|
llvm::SmallPtrSet<VarDecl*, 8> &Decls;
|
2013-01-13 03:30:44 +08:00
|
|
|
SmallVector<SourceRange, 10> &Ranges;
|
2012-05-01 02:01:30 +08:00
|
|
|
bool Simple;
|
|
|
|
public:
|
|
|
|
typedef EvaluatedExprVisitor<DeclExtractor> Inherited;
|
|
|
|
|
|
|
|
DeclExtractor(Sema &S, llvm::SmallPtrSet<VarDecl*, 8> &Decls,
|
2013-01-13 03:30:44 +08:00
|
|
|
SmallVector<SourceRange, 10> &Ranges) :
|
2012-05-01 02:01:30 +08:00
|
|
|
Inherited(S.Context),
|
|
|
|
Decls(Decls),
|
|
|
|
Ranges(Ranges),
|
2012-06-07 01:32:50 +08:00
|
|
|
Simple(true) {}
|
2012-05-01 02:01:30 +08:00
|
|
|
|
|
|
|
bool isSimple() { return Simple; }
|
|
|
|
|
|
|
|
// Replaces the method in EvaluatedExprVisitor.
|
|
|
|
void VisitMemberExpr(MemberExpr* E) {
|
|
|
|
Simple = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Any Stmt not whitelisted will cause the condition to be marked complex.
|
|
|
|
void VisitStmt(Stmt *S) {
|
|
|
|
Simple = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitBinaryOperator(BinaryOperator *E) {
|
|
|
|
Visit(E->getLHS());
|
|
|
|
Visit(E->getRHS());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitCastExpr(CastExpr *E) {
|
|
|
|
Visit(E->getSubExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitUnaryOperator(UnaryOperator *E) {
|
|
|
|
// Skip checking conditionals with derefernces.
|
|
|
|
if (E->getOpcode() == UO_Deref)
|
|
|
|
Simple = false;
|
|
|
|
else
|
|
|
|
Visit(E->getSubExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitConditionalOperator(ConditionalOperator *E) {
|
|
|
|
Visit(E->getCond());
|
|
|
|
Visit(E->getTrueExpr());
|
|
|
|
Visit(E->getFalseExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitParenExpr(ParenExpr *E) {
|
|
|
|
Visit(E->getSubExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitBinaryConditionalOperator(BinaryConditionalOperator *E) {
|
|
|
|
Visit(E->getOpaqueValue()->getSourceExpr());
|
|
|
|
Visit(E->getFalseExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitIntegerLiteral(IntegerLiteral *E) { }
|
|
|
|
void VisitFloatingLiteral(FloatingLiteral *E) { }
|
|
|
|
void VisitCXXBoolLiteralExpr(CXXBoolLiteralExpr *E) { }
|
|
|
|
void VisitCharacterLiteral(CharacterLiteral *E) { }
|
|
|
|
void VisitGNUNullExpr(GNUNullExpr *E) { }
|
|
|
|
void VisitImaginaryLiteral(ImaginaryLiteral *E) { }
|
|
|
|
|
|
|
|
void VisitDeclRefExpr(DeclRefExpr *E) {
|
|
|
|
VarDecl *VD = dyn_cast<VarDecl>(E->getDecl());
|
|
|
|
if (!VD) return;
|
|
|
|
|
|
|
|
Ranges.push_back(E->getSourceRange());
|
|
|
|
|
|
|
|
Decls.insert(VD);
|
|
|
|
}
|
|
|
|
|
|
|
|
}; // end class DeclExtractor
|
|
|
|
|
|
|
|
// DeclMatcher checks to see if the decls are used in a non-evauluated
|
2012-08-11 01:56:09 +08:00
|
|
|
// context.
|
2012-05-01 02:01:30 +08:00
|
|
|
class DeclMatcher : public EvaluatedExprVisitor<DeclMatcher> {
|
|
|
|
llvm::SmallPtrSet<VarDecl*, 8> &Decls;
|
|
|
|
bool FoundDecl;
|
|
|
|
|
|
|
|
public:
|
|
|
|
typedef EvaluatedExprVisitor<DeclMatcher> Inherited;
|
|
|
|
|
|
|
|
DeclMatcher(Sema &S, llvm::SmallPtrSet<VarDecl*, 8> &Decls, Stmt *Statement) :
|
|
|
|
Inherited(S.Context), Decls(Decls), FoundDecl(false) {
|
|
|
|
if (!Statement) return;
|
|
|
|
|
|
|
|
Visit(Statement);
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitReturnStmt(ReturnStmt *S) {
|
|
|
|
FoundDecl = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitBreakStmt(BreakStmt *S) {
|
|
|
|
FoundDecl = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitGotoStmt(GotoStmt *S) {
|
|
|
|
FoundDecl = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitCastExpr(CastExpr *E) {
|
|
|
|
if (E->getCastKind() == CK_LValueToRValue)
|
|
|
|
CheckLValueToRValueCast(E->getSubExpr());
|
|
|
|
else
|
|
|
|
Visit(E->getSubExpr());
|
|
|
|
}
|
|
|
|
|
|
|
|
void CheckLValueToRValueCast(Expr *E) {
|
|
|
|
E = E->IgnoreParenImpCasts();
|
|
|
|
|
|
|
|
if (isa<DeclRefExpr>(E)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (ConditionalOperator *CO = dyn_cast<ConditionalOperator>(E)) {
|
|
|
|
Visit(CO->getCond());
|
|
|
|
CheckLValueToRValueCast(CO->getTrueExpr());
|
|
|
|
CheckLValueToRValueCast(CO->getFalseExpr());
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (BinaryConditionalOperator *BCO =
|
|
|
|
dyn_cast<BinaryConditionalOperator>(E)) {
|
|
|
|
CheckLValueToRValueCast(BCO->getOpaqueValue()->getSourceExpr());
|
|
|
|
CheckLValueToRValueCast(BCO->getFalseExpr());
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
Visit(E);
|
|
|
|
}
|
|
|
|
|
|
|
|
void VisitDeclRefExpr(DeclRefExpr *E) {
|
|
|
|
if (VarDecl *VD = dyn_cast<VarDecl>(E->getDecl()))
|
|
|
|
if (Decls.count(VD))
|
|
|
|
FoundDecl = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool FoundDeclInUse() { return FoundDecl; }
|
|
|
|
|
|
|
|
}; // end class DeclMatcher
|
|
|
|
|
|
|
|
void CheckForLoopConditionalStatement(Sema &S, Expr *Second,
|
|
|
|
Expr *Third, Stmt *Body) {
|
|
|
|
// Condition is empty
|
|
|
|
if (!Second) return;
|
|
|
|
|
|
|
|
if (S.Diags.getDiagnosticLevel(diag::warn_variables_not_in_loop_body,
|
|
|
|
Second->getLocStart())
|
|
|
|
== DiagnosticsEngine::Ignored)
|
|
|
|
return;
|
|
|
|
|
|
|
|
PartialDiagnostic PDiag = S.PDiag(diag::warn_variables_not_in_loop_body);
|
|
|
|
llvm::SmallPtrSet<VarDecl*, 8> Decls;
|
2013-01-13 03:30:44 +08:00
|
|
|
SmallVector<SourceRange, 10> Ranges;
|
2012-06-07 01:32:50 +08:00
|
|
|
DeclExtractor DE(S, Decls, Ranges);
|
2012-05-01 02:01:30 +08:00
|
|
|
DE.Visit(Second);
|
|
|
|
|
|
|
|
// Don't analyze complex conditionals.
|
|
|
|
if (!DE.isSimple()) return;
|
|
|
|
|
|
|
|
// No decls found.
|
|
|
|
if (Decls.size() == 0) return;
|
|
|
|
|
2012-05-04 11:01:54 +08:00
|
|
|
// Don't warn on volatile, static, or global variables.
|
2012-05-01 02:01:30 +08:00
|
|
|
for (llvm::SmallPtrSet<VarDecl*, 8>::iterator I = Decls.begin(),
|
|
|
|
E = Decls.end();
|
|
|
|
I != E; ++I)
|
2012-05-04 11:01:54 +08:00
|
|
|
if ((*I)->getType().isVolatileQualified() ||
|
|
|
|
(*I)->hasGlobalStorage()) return;
|
2012-05-01 02:01:30 +08:00
|
|
|
|
|
|
|
if (DeclMatcher(S, Decls, Second).FoundDeclInUse() ||
|
|
|
|
DeclMatcher(S, Decls, Third).FoundDeclInUse() ||
|
|
|
|
DeclMatcher(S, Decls, Body).FoundDeclInUse())
|
|
|
|
return;
|
|
|
|
|
|
|
|
// Load decl names into diagnostic.
|
|
|
|
if (Decls.size() > 4)
|
|
|
|
PDiag << 0;
|
|
|
|
else {
|
|
|
|
PDiag << Decls.size();
|
|
|
|
for (llvm::SmallPtrSet<VarDecl*, 8>::iterator I = Decls.begin(),
|
|
|
|
E = Decls.end();
|
|
|
|
I != E; ++I)
|
|
|
|
PDiag << (*I)->getDeclName();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Load SourceRanges into diagnostic if there is room.
|
|
|
|
// Otherwise, load the SourceRange of the conditional expression.
|
|
|
|
if (Ranges.size() <= PartialDiagnostic::MaxArguments)
|
2013-01-13 03:30:44 +08:00
|
|
|
for (SmallVector<SourceRange, 10>::iterator I = Ranges.begin(),
|
|
|
|
E = Ranges.end();
|
2012-05-01 02:01:30 +08:00
|
|
|
I != E; ++I)
|
|
|
|
PDiag << *I;
|
|
|
|
else
|
|
|
|
PDiag << Second->getSourceRange();
|
|
|
|
|
|
|
|
S.Diag(Ranges.begin()->getBegin(), PDiag);
|
|
|
|
}
|
|
|
|
|
|
|
|
} // end namespace
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2009-01-17 07:28:06 +08:00
|
|
|
Sema::ActOnForStmt(SourceLocation ForLoc, SourceLocation LParenLoc,
|
2010-08-24 07:25:46 +08:00
|
|
|
Stmt *First, FullExprArg second, Decl *secondVar,
|
2009-11-25 08:27:52 +08:00
|
|
|
FullExprArg third,
|
2010-08-24 07:25:46 +08:00
|
|
|
SourceLocation RParenLoc, Stmt *Body) {
|
2012-03-11 15:00:24 +08:00
|
|
|
if (!getLangOpts().CPlusPlus) {
|
2008-09-10 10:17:11 +08:00
|
|
|
if (DeclStmt *DS = dyn_cast_or_null<DeclStmt>(First)) {
|
2008-11-20 14:38:18 +08:00
|
|
|
// C99 6.8.5p3: The declaration part of a 'for' statement shall only
|
|
|
|
// declare identifiers for objects having storage class 'auto' or
|
|
|
|
// 'register'.
|
2008-09-10 10:17:11 +08:00
|
|
|
for (DeclStmt::decl_iterator DI=DS->decl_begin(), DE=DS->decl_end();
|
|
|
|
DI!=DE; ++DI) {
|
|
|
|
VarDecl *VD = dyn_cast<VarDecl>(*DI);
|
2010-10-15 12:57:14 +08:00
|
|
|
if (VD && VD->isLocalVarDecl() && !VD->hasLocalStorage())
|
2008-09-10 10:17:11 +08:00
|
|
|
VD = 0;
|
2013-04-09 04:52:24 +08:00
|
|
|
if (VD == 0) {
|
|
|
|
Diag((*DI)->getLocation(), diag::err_non_local_variable_decl_in_for);
|
|
|
|
(*DI)->setInvalidDecl();
|
|
|
|
}
|
2008-09-10 10:17:11 +08:00
|
|
|
}
|
2007-08-28 13:03:08 +08:00
|
|
|
}
|
2007-05-29 10:14:17 +08:00
|
|
|
}
|
2009-11-25 08:27:52 +08:00
|
|
|
|
2012-05-01 02:01:30 +08:00
|
|
|
CheckForLoopConditionalStatement(*this, second.get(), third.get(), Body);
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
ExprResult SecondResult(second.release());
|
2009-11-25 08:27:52 +08:00
|
|
|
VarDecl *ConditionVar = 0;
|
2010-08-21 17:40:31 +08:00
|
|
|
if (secondVar) {
|
|
|
|
ConditionVar = cast<VarDecl>(secondVar);
|
2010-05-07 01:25:47 +08:00
|
|
|
SecondResult = CheckConditionVariable(ConditionVar, ForLoc, true);
|
2009-11-25 08:27:52 +08:00
|
|
|
if (SecondResult.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2009-11-25 08:27:52 +08:00
|
|
|
Expr *Third = third.release().takeAs<Expr>();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2009-08-01 09:39:59 +08:00
|
|
|
DiagnoseUnusedExprResult(First);
|
|
|
|
DiagnoseUnusedExprResult(Third);
|
2009-07-31 06:39:03 +08:00
|
|
|
DiagnoseUnusedExprResult(Body);
|
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
if (isa<NullStmt>(Body))
|
|
|
|
getCurCompoundScope().setHasEmptyLoopBodies();
|
|
|
|
|
2011-01-27 15:10:08 +08:00
|
|
|
return Owned(new (Context) ForStmt(Context, First,
|
|
|
|
SecondResult.take(), ConditionVar,
|
|
|
|
Third, Body, ForLoc, LParenLoc,
|
2010-06-22 07:44:13 +08:00
|
|
|
RParenLoc));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-12-04 11:47:34 +08:00
|
|
|
/// In an Objective C collection iteration statement:
|
|
|
|
/// for (x in y)
|
|
|
|
/// x can be an arbitrary l-value expression. Bind it up as a
|
|
|
|
/// full-expression.
|
|
|
|
StmtResult Sema::ActOnForEachLValueExpr(Expr *E) {
|
2012-03-30 13:43:39 +08:00
|
|
|
// Reduce placeholder expressions here. Note that this rejects the
|
|
|
|
// use of pseudo-object l-values in this position.
|
|
|
|
ExprResult result = CheckPlaceholderExpr(E);
|
|
|
|
if (result.isInvalid()) return StmtError();
|
|
|
|
E = result.take();
|
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
ExprResult FullExpr = ActOnFinishFullExpr(E);
|
|
|
|
if (FullExpr.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
return StmtResult(static_cast<Stmt*>(FullExpr.take()));
|
2010-12-04 11:47:34 +08:00
|
|
|
}
|
|
|
|
|
2011-07-27 09:07:15 +08:00
|
|
|
ExprResult
|
2012-07-04 06:00:52 +08:00
|
|
|
Sema::CheckObjCForCollectionOperand(SourceLocation forLoc, Expr *collection) {
|
|
|
|
if (!collection)
|
|
|
|
return ExprError();
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2011-07-27 09:07:15 +08:00
|
|
|
// Bail out early if we've got a type-dependent expression.
|
|
|
|
if (collection->isTypeDependent()) return Owned(collection);
|
|
|
|
|
|
|
|
// Perform normal l-value conversion.
|
|
|
|
ExprResult result = DefaultFunctionArrayLvalueConversion(collection);
|
|
|
|
if (result.isInvalid())
|
|
|
|
return ExprError();
|
|
|
|
collection = result.take();
|
|
|
|
|
|
|
|
// The operand needs to have object-pointer type.
|
|
|
|
// TODO: should we do a contextual conversion?
|
|
|
|
const ObjCObjectPointerType *pointerType =
|
|
|
|
collection->getType()->getAs<ObjCObjectPointerType>();
|
|
|
|
if (!pointerType)
|
|
|
|
return Diag(forLoc, diag::err_collection_expr_type)
|
|
|
|
<< collection->getType() << collection->getSourceRange();
|
|
|
|
|
|
|
|
// Check that the operand provides
|
|
|
|
// - countByEnumeratingWithState:objects:count:
|
|
|
|
const ObjCObjectType *objectType = pointerType->getObjectType();
|
|
|
|
ObjCInterfaceDecl *iface = objectType->getInterface();
|
|
|
|
|
|
|
|
// If we have a forward-declared type, we can't do this check.
|
2011-11-15 06:10:01 +08:00
|
|
|
// Under ARC, it is an error not to have a forward-declared class.
|
2012-08-11 01:56:09 +08:00
|
|
|
if (iface &&
|
2011-11-15 06:10:01 +08:00
|
|
|
RequireCompleteType(forLoc, QualType(objectType, 0),
|
2012-03-11 15:00:24 +08:00
|
|
|
getLangOpts().ObjCAutoRefCount
|
2012-05-05 00:32:21 +08:00
|
|
|
? diag::err_arc_collection_forward
|
|
|
|
: 0,
|
|
|
|
collection)) {
|
2011-07-27 09:07:15 +08:00
|
|
|
// Otherwise, if we have any useful type information, check that
|
|
|
|
// the type declares the appropriate method.
|
|
|
|
} else if (iface || !objectType->qual_empty()) {
|
|
|
|
IdentifierInfo *selectorIdents[] = {
|
|
|
|
&Context.Idents.get("countByEnumeratingWithState"),
|
|
|
|
&Context.Idents.get("objects"),
|
|
|
|
&Context.Idents.get("count")
|
|
|
|
};
|
|
|
|
Selector selector = Context.Selectors.getSelector(3, &selectorIdents[0]);
|
|
|
|
|
|
|
|
ObjCMethodDecl *method = 0;
|
|
|
|
|
|
|
|
// If there's an interface, look in both the public and private APIs.
|
|
|
|
if (iface) {
|
|
|
|
method = iface->lookupInstanceMethod(selector);
|
2012-07-28 03:07:44 +08:00
|
|
|
if (!method) method = iface->lookupPrivateMethod(selector);
|
2011-07-27 09:07:15 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// Also check protocol qualifiers.
|
|
|
|
if (!method)
|
|
|
|
method = LookupMethodInQualifiedType(selector, pointerType,
|
|
|
|
/*instance*/ true);
|
|
|
|
|
|
|
|
// If we didn't find it anywhere, give up.
|
|
|
|
if (!method) {
|
|
|
|
Diag(forLoc, diag::warn_collection_expr_type)
|
|
|
|
<< collection->getType() << selector << collection->getSourceRange();
|
|
|
|
}
|
|
|
|
|
|
|
|
// TODO: check for an incompatible signature?
|
|
|
|
}
|
|
|
|
|
|
|
|
// Wrap up any cleanups in the expression.
|
2013-01-15 06:39:08 +08:00
|
|
|
return Owned(collection);
|
2011-07-27 09:07:15 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2009-01-17 07:28:06 +08:00
|
|
|
Sema::ActOnObjCForCollectionStmt(SourceLocation ForLoc,
|
2012-07-04 06:00:52 +08:00
|
|
|
Stmt *First, Expr *collection,
|
|
|
|
SourceLocation RParenLoc) {
|
2012-08-11 01:56:09 +08:00
|
|
|
|
|
|
|
ExprResult CollectionExprResult =
|
2012-07-04 06:00:52 +08:00
|
|
|
CheckObjCForCollectionOperand(ForLoc, collection);
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2008-01-11 04:33:58 +08:00
|
|
|
if (First) {
|
|
|
|
QualType FirstType;
|
|
|
|
if (DeclStmt *DS = dyn_cast<DeclStmt>(First)) {
|
2009-03-28 14:33:19 +08:00
|
|
|
if (!DS->isSingleDecl())
|
2009-01-17 07:28:06 +08:00
|
|
|
return StmtError(Diag((*DS->decl_begin())->getLocation(),
|
|
|
|
diag::err_toomany_element_decls));
|
|
|
|
|
2013-04-09 04:52:24 +08:00
|
|
|
VarDecl *D = dyn_cast<VarDecl>(DS->getSingleDecl());
|
|
|
|
if (!D || D->isInvalidDecl())
|
|
|
|
return StmtError();
|
|
|
|
|
2011-06-16 07:02:42 +08:00
|
|
|
FirstType = D->getType();
|
2008-11-20 14:38:18 +08:00
|
|
|
// C99 6.8.5p3: The declaration part of a 'for' statement shall only
|
|
|
|
// declare identifiers for objects having storage class 'auto' or
|
|
|
|
// 'register'.
|
2011-06-16 07:02:42 +08:00
|
|
|
if (!D->hasLocalStorage())
|
|
|
|
return StmtError(Diag(D->getLocation(),
|
2013-04-09 04:52:24 +08:00
|
|
|
diag::err_non_local_variable_decl_in_for));
|
2013-04-09 02:25:02 +08:00
|
|
|
|
|
|
|
// If the type contained 'auto', deduce the 'auto' to 'id'.
|
|
|
|
if (FirstType->getContainedAutoType()) {
|
|
|
|
TypeSourceInfo *DeducedType = 0;
|
|
|
|
OpaqueValueExpr OpaqueId(D->getLocation(), Context.getObjCIdType(),
|
|
|
|
VK_RValue);
|
|
|
|
Expr *DeducedInit = &OpaqueId;
|
|
|
|
if (DeduceAutoType(D->getTypeSourceInfo(), DeducedInit, DeducedType)
|
|
|
|
== DAR_Failed) {
|
|
|
|
DiagnoseAutoDeductionFailure(D, DeducedInit);
|
|
|
|
}
|
|
|
|
if (!DeducedType) {
|
|
|
|
D->setInvalidDecl();
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
|
|
|
D->setTypeSourceInfo(DeducedType);
|
|
|
|
D->setType(DeducedType->getType());
|
|
|
|
FirstType = DeducedType->getType();
|
|
|
|
|
|
|
|
if (ActiveTemplateInstantiations.empty()) {
|
|
|
|
SourceLocation Loc = DeducedType->getTypeLoc().getBeginLoc();
|
|
|
|
Diag(Loc, diag::warn_auto_var_is_id)
|
|
|
|
<< D->getDeclName();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-08-26 02:16:36 +08:00
|
|
|
} else {
|
2010-04-23 07:10:45 +08:00
|
|
|
Expr *FirstE = cast<Expr>(First);
|
2010-11-24 13:12:34 +08:00
|
|
|
if (!FirstE->isTypeDependent() && !FirstE->isLValue())
|
2009-01-17 07:28:06 +08:00
|
|
|
return StmtError(Diag(First->getLocStart(),
|
|
|
|
diag::err_selector_element_not_lvalue)
|
|
|
|
<< First->getSourceRange());
|
2008-08-26 02:16:36 +08:00
|
|
|
|
2009-09-09 23:08:12 +08:00
|
|
|
FirstType = static_cast<Expr*>(First)->getType();
|
2008-08-26 02:16:36 +08:00
|
|
|
}
|
2010-04-23 07:10:45 +08:00
|
|
|
if (!FirstType->isDependentType() &&
|
|
|
|
!FirstType->isObjCObjectPointerType() &&
|
2009-08-15 05:53:27 +08:00
|
|
|
!FirstType->isBlockPointerType())
|
2012-07-04 06:00:52 +08:00
|
|
|
return StmtError(Diag(ForLoc, diag::err_selector_element_type)
|
|
|
|
<< FirstType << First->getSourceRange());
|
2008-01-04 01:55:25 +08:00
|
|
|
}
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
if (CollectionExprResult.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
CollectionExprResult = ActOnFinishFullExpr(CollectionExprResult.take());
|
2012-07-04 06:00:52 +08:00
|
|
|
if (CollectionExprResult.isInvalid())
|
|
|
|
return StmtError();
|
2012-08-11 01:56:09 +08:00
|
|
|
|
|
|
|
return Owned(new (Context) ObjCForCollectionStmt(First,
|
|
|
|
CollectionExprResult.take(), 0,
|
2009-02-07 09:47:29 +08:00
|
|
|
ForLoc, RParenLoc));
|
2008-01-04 01:55:25 +08:00
|
|
|
}
|
2006-11-10 13:07:45 +08:00
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
/// Finish building a variable declaration for a for-range statement.
|
|
|
|
/// \return true if an error occurs.
|
|
|
|
static bool FinishForRangeVarDecl(Sema &SemaRef, VarDecl *Decl, Expr *Init,
|
|
|
|
SourceLocation Loc, int diag) {
|
|
|
|
// Deduce the type for the iterator variable now rather than leaving it to
|
|
|
|
// AddInitializerToDecl, so we can produce a more suitable diagnostic.
|
|
|
|
TypeSourceInfo *InitTSI = 0;
|
2012-01-18 06:50:08 +08:00
|
|
|
if ((!isa<InitListExpr>(Init) && Init->getType()->isVoidType()) ||
|
2012-01-24 06:09:39 +08:00
|
|
|
SemaRef.DeduceAutoType(Decl->getTypeSourceInfo(), Init, InitTSI) ==
|
|
|
|
Sema::DAR_Failed)
|
2011-04-15 06:09:26 +08:00
|
|
|
SemaRef.Diag(Loc, diag) << Init->getType();
|
|
|
|
if (!InitTSI) {
|
|
|
|
Decl->setInvalidDecl();
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
Decl->setTypeSourceInfo(InitTSI);
|
|
|
|
Decl->setType(InitTSI->getType());
|
|
|
|
|
2011-06-16 07:02:42 +08:00
|
|
|
// In ARC, infer lifetime.
|
|
|
|
// FIXME: ARC may want to turn this into 'const __unsafe_unretained' if
|
|
|
|
// we're doing the equivalent of fast iteration.
|
2012-08-11 01:56:09 +08:00
|
|
|
if (SemaRef.getLangOpts().ObjCAutoRefCount &&
|
2011-06-16 07:02:42 +08:00
|
|
|
SemaRef.inferObjCARCLifetime(Decl))
|
|
|
|
Decl->setInvalidDecl();
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
SemaRef.AddInitializerToDecl(Decl, Init, /*DirectInit=*/false,
|
|
|
|
/*TypeMayContainAuto=*/false);
|
|
|
|
SemaRef.FinalizeDeclaration(Decl);
|
2011-04-18 23:49:25 +08:00
|
|
|
SemaRef.CurContext->addHiddenDecl(Decl);
|
2011-04-15 06:09:26 +08:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2012-08-21 08:52:01 +08:00
|
|
|
namespace {
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
/// Produce a note indicating which begin/end function was implicitly called
|
2012-08-21 08:52:01 +08:00
|
|
|
/// by a C++11 for-range statement. This is often not obvious from the code,
|
2011-04-15 06:09:26 +08:00
|
|
|
/// nor from the diagnostics produced when analysing the implicit expressions
|
|
|
|
/// required in a for-range statement.
|
|
|
|
void NoteForRangeBeginEndFunction(Sema &SemaRef, Expr *E,
|
2012-08-21 08:52:01 +08:00
|
|
|
Sema::BeginEndFunction BEF) {
|
2011-04-15 06:09:26 +08:00
|
|
|
CallExpr *CE = dyn_cast<CallExpr>(E);
|
|
|
|
if (!CE)
|
|
|
|
return;
|
|
|
|
FunctionDecl *D = dyn_cast<FunctionDecl>(CE->getCalleeDecl());
|
|
|
|
if (!D)
|
|
|
|
return;
|
|
|
|
SourceLocation Loc = D->getLocation();
|
|
|
|
|
|
|
|
std::string Description;
|
|
|
|
bool IsTemplate = false;
|
|
|
|
if (FunctionTemplateDecl *FunTmpl = D->getPrimaryTemplate()) {
|
|
|
|
Description = SemaRef.getTemplateArgumentBindingsText(
|
|
|
|
FunTmpl->getTemplateParameters(), *D->getTemplateSpecializationArgs());
|
|
|
|
IsTemplate = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
SemaRef.Diag(Loc, diag::note_for_range_begin_end)
|
|
|
|
<< BEF << IsTemplate << Description << E->getType();
|
|
|
|
}
|
|
|
|
|
2012-08-21 08:52:01 +08:00
|
|
|
/// Build a variable declaration for a for-range statement.
|
|
|
|
VarDecl *BuildForRangeVarDecl(Sema &SemaRef, SourceLocation Loc,
|
|
|
|
QualType Type, const char *Name) {
|
|
|
|
DeclContext *DC = SemaRef.CurContext;
|
|
|
|
IdentifierInfo *II = &SemaRef.PP.getIdentifierTable().get(Name);
|
|
|
|
TypeSourceInfo *TInfo = SemaRef.Context.getTrivialTypeSourceInfo(Type, Loc);
|
|
|
|
VarDecl *Decl = VarDecl::Create(SemaRef.Context, DC, Loc, Loc, II, Type,
|
2013-04-04 03:27:57 +08:00
|
|
|
TInfo, SC_None);
|
2012-08-21 08:52:01 +08:00
|
|
|
Decl->setImplicit();
|
|
|
|
return Decl;
|
2011-04-15 06:09:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
}
|
|
|
|
|
2012-07-07 03:04:04 +08:00
|
|
|
static bool ObjCEnumerationCollection(Expr *Collection) {
|
|
|
|
return !Collection->isTypeDependent()
|
|
|
|
&& Collection->getType()->getAs<ObjCObjectPointerType>() != 0;
|
|
|
|
}
|
|
|
|
|
2012-08-17 05:47:25 +08:00
|
|
|
/// ActOnCXXForRangeStmt - Check and build a C++11 for-range statement.
|
2011-04-15 06:09:26 +08:00
|
|
|
///
|
2012-08-17 05:47:25 +08:00
|
|
|
/// C++11 [stmt.ranged]:
|
2011-04-15 06:09:26 +08:00
|
|
|
/// A range-based for statement is equivalent to
|
|
|
|
///
|
|
|
|
/// {
|
|
|
|
/// auto && __range = range-init;
|
|
|
|
/// for ( auto __begin = begin-expr,
|
|
|
|
/// __end = end-expr;
|
|
|
|
/// __begin != __end;
|
|
|
|
/// ++__begin ) {
|
|
|
|
/// for-range-declaration = *__begin;
|
|
|
|
/// statement
|
|
|
|
/// }
|
|
|
|
/// }
|
|
|
|
///
|
|
|
|
/// The body of the loop is not available yet, since it cannot be analysed until
|
|
|
|
/// we have determined the type of the for-range-declaration.
|
|
|
|
StmtResult
|
2012-08-17 05:47:25 +08:00
|
|
|
Sema::ActOnCXXForRangeStmt(SourceLocation ForLoc,
|
2011-04-15 06:09:26 +08:00
|
|
|
Stmt *First, SourceLocation ColonLoc, Expr *Range,
|
2012-09-21 05:52:32 +08:00
|
|
|
SourceLocation RParenLoc, BuildForRangeKind Kind) {
|
2011-04-15 06:09:26 +08:00
|
|
|
if (!First || !Range)
|
|
|
|
return StmtError();
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-07-07 03:04:04 +08:00
|
|
|
if (ObjCEnumerationCollection(Range))
|
2012-08-17 05:47:25 +08:00
|
|
|
return ActOnObjCForCollectionStmt(ForLoc, First, Range, RParenLoc);
|
2011-04-15 06:09:26 +08:00
|
|
|
|
|
|
|
DeclStmt *DS = dyn_cast<DeclStmt>(First);
|
|
|
|
assert(DS && "first part of for range not a decl stmt");
|
|
|
|
|
|
|
|
if (!DS->isSingleDecl()) {
|
|
|
|
Diag(DS->getStartLoc(), diag::err_type_defined_in_for_range);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
if (DS->getSingleDecl()->isInvalidDecl())
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
if (DiagnoseUnexpandedParameterPack(Range, UPPC_Expression))
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
// Build auto && __range = range-init
|
|
|
|
SourceLocation RangeLoc = Range->getLocStart();
|
|
|
|
VarDecl *RangeVar = BuildForRangeVarDecl(*this, RangeLoc,
|
|
|
|
Context.getAutoRRefDeductType(),
|
|
|
|
"__range");
|
|
|
|
if (FinishForRangeVarDecl(*this, RangeVar, Range, RangeLoc,
|
|
|
|
diag::err_for_range_deduction_failure))
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
// Claim the type doesn't contain auto: we've already done the checking.
|
|
|
|
DeclGroupPtrTy RangeGroup =
|
|
|
|
BuildDeclaratorGroup((Decl**)&RangeVar, 1, /*TypeMayContainAuto=*/false);
|
|
|
|
StmtResult RangeDecl = ActOnDeclStmt(RangeGroup, RangeLoc, RangeLoc);
|
|
|
|
if (RangeDecl.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
return BuildCXXForRangeStmt(ForLoc, ColonLoc, RangeDecl.get(),
|
|
|
|
/*BeginEndDecl=*/0, /*Cond=*/0, /*Inc=*/0, DS,
|
2012-09-21 05:52:32 +08:00
|
|
|
RParenLoc, Kind);
|
2012-08-21 08:52:01 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/// \brief Create the initialization, compare, and increment steps for
|
|
|
|
/// the range-based for loop expression.
|
|
|
|
/// This function does not handle array-based for loops,
|
|
|
|
/// which are created in Sema::BuildCXXForRangeStmt.
|
|
|
|
///
|
|
|
|
/// \returns a ForRangeStatus indicating success or what kind of error occurred.
|
|
|
|
/// BeginExpr and EndExpr are set and FRS_Success is returned on success;
|
|
|
|
/// CandidateSet and BEF are set and some non-success value is returned on
|
|
|
|
/// failure.
|
|
|
|
static Sema::ForRangeStatus BuildNonArrayForRange(Sema &SemaRef, Scope *S,
|
|
|
|
Expr *BeginRange, Expr *EndRange,
|
|
|
|
QualType RangeType,
|
|
|
|
VarDecl *BeginVar,
|
|
|
|
VarDecl *EndVar,
|
|
|
|
SourceLocation ColonLoc,
|
|
|
|
OverloadCandidateSet *CandidateSet,
|
|
|
|
ExprResult *BeginExpr,
|
|
|
|
ExprResult *EndExpr,
|
|
|
|
Sema::BeginEndFunction *BEF) {
|
|
|
|
DeclarationNameInfo BeginNameInfo(
|
|
|
|
&SemaRef.PP.getIdentifierTable().get("begin"), ColonLoc);
|
|
|
|
DeclarationNameInfo EndNameInfo(&SemaRef.PP.getIdentifierTable().get("end"),
|
|
|
|
ColonLoc);
|
|
|
|
|
|
|
|
LookupResult BeginMemberLookup(SemaRef, BeginNameInfo,
|
|
|
|
Sema::LookupMemberName);
|
|
|
|
LookupResult EndMemberLookup(SemaRef, EndNameInfo, Sema::LookupMemberName);
|
|
|
|
|
|
|
|
if (CXXRecordDecl *D = RangeType->getAsCXXRecordDecl()) {
|
|
|
|
// - if _RangeT is a class type, the unqualified-ids begin and end are
|
|
|
|
// looked up in the scope of class _RangeT as if by class member access
|
|
|
|
// lookup (3.4.5), and if either (or both) finds at least one
|
|
|
|
// declaration, begin-expr and end-expr are __range.begin() and
|
|
|
|
// __range.end(), respectively;
|
|
|
|
SemaRef.LookupQualifiedName(BeginMemberLookup, D);
|
|
|
|
SemaRef.LookupQualifiedName(EndMemberLookup, D);
|
|
|
|
|
|
|
|
if (BeginMemberLookup.empty() != EndMemberLookup.empty()) {
|
|
|
|
SourceLocation RangeLoc = BeginVar->getLocation();
|
|
|
|
*BEF = BeginMemberLookup.empty() ? Sema::BEF_end : Sema::BEF_begin;
|
|
|
|
|
|
|
|
SemaRef.Diag(RangeLoc, diag::err_for_range_member_begin_end_mismatch)
|
|
|
|
<< RangeLoc << BeginRange->getType() << *BEF;
|
|
|
|
return Sema::FRS_DiagnosticIssued;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
// - otherwise, begin-expr and end-expr are begin(__range) and
|
|
|
|
// end(__range), respectively, where begin and end are looked up with
|
|
|
|
// argument-dependent lookup (3.4.2). For the purposes of this name
|
|
|
|
// lookup, namespace std is an associated namespace.
|
|
|
|
|
|
|
|
}
|
|
|
|
|
|
|
|
*BEF = Sema::BEF_begin;
|
|
|
|
Sema::ForRangeStatus RangeStatus =
|
|
|
|
SemaRef.BuildForRangeBeginEndCall(S, ColonLoc, ColonLoc, BeginVar,
|
|
|
|
Sema::BEF_begin, BeginNameInfo,
|
|
|
|
BeginMemberLookup, CandidateSet,
|
|
|
|
BeginRange, BeginExpr);
|
|
|
|
|
|
|
|
if (RangeStatus != Sema::FRS_Success)
|
|
|
|
return RangeStatus;
|
|
|
|
if (FinishForRangeVarDecl(SemaRef, BeginVar, BeginExpr->get(), ColonLoc,
|
|
|
|
diag::err_for_range_iter_deduction_failure)) {
|
|
|
|
NoteForRangeBeginEndFunction(SemaRef, BeginExpr->get(), *BEF);
|
|
|
|
return Sema::FRS_DiagnosticIssued;
|
|
|
|
}
|
|
|
|
|
|
|
|
*BEF = Sema::BEF_end;
|
|
|
|
RangeStatus =
|
|
|
|
SemaRef.BuildForRangeBeginEndCall(S, ColonLoc, ColonLoc, EndVar,
|
|
|
|
Sema::BEF_end, EndNameInfo,
|
|
|
|
EndMemberLookup, CandidateSet,
|
|
|
|
EndRange, EndExpr);
|
|
|
|
if (RangeStatus != Sema::FRS_Success)
|
|
|
|
return RangeStatus;
|
|
|
|
if (FinishForRangeVarDecl(SemaRef, EndVar, EndExpr->get(), ColonLoc,
|
|
|
|
diag::err_for_range_iter_deduction_failure)) {
|
|
|
|
NoteForRangeBeginEndFunction(SemaRef, EndExpr->get(), *BEF);
|
|
|
|
return Sema::FRS_DiagnosticIssued;
|
|
|
|
}
|
|
|
|
return Sema::FRS_Success;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Speculatively attempt to dereference an invalid range expression.
|
2012-09-21 05:52:32 +08:00
|
|
|
/// If the attempt fails, this function will return a valid, null StmtResult
|
|
|
|
/// and emit no diagnostics.
|
2012-08-21 08:52:01 +08:00
|
|
|
static StmtResult RebuildForRangeWithDereference(Sema &SemaRef, Scope *S,
|
|
|
|
SourceLocation ForLoc,
|
|
|
|
Stmt *LoopVarDecl,
|
|
|
|
SourceLocation ColonLoc,
|
|
|
|
Expr *Range,
|
|
|
|
SourceLocation RangeLoc,
|
|
|
|
SourceLocation RParenLoc) {
|
2012-09-21 05:52:32 +08:00
|
|
|
// Determine whether we can rebuild the for-range statement with a
|
|
|
|
// dereferenced range expression.
|
|
|
|
ExprResult AdjustedRange;
|
|
|
|
{
|
|
|
|
Sema::SFINAETrap Trap(SemaRef);
|
|
|
|
|
|
|
|
AdjustedRange = SemaRef.BuildUnaryOp(S, RangeLoc, UO_Deref, Range);
|
|
|
|
if (AdjustedRange.isInvalid())
|
|
|
|
return StmtResult();
|
|
|
|
|
|
|
|
StmtResult SR =
|
|
|
|
SemaRef.ActOnCXXForRangeStmt(ForLoc, LoopVarDecl, ColonLoc,
|
|
|
|
AdjustedRange.get(), RParenLoc,
|
|
|
|
Sema::BFRK_Check);
|
|
|
|
if (SR.isInvalid())
|
|
|
|
return StmtResult();
|
|
|
|
}
|
|
|
|
|
|
|
|
// The attempt to dereference worked well enough that it could produce a valid
|
|
|
|
// loop. Produce a fixit, and rebuild the loop with diagnostics enabled, in
|
|
|
|
// case there are any other (non-fatal) problems with it.
|
|
|
|
SemaRef.Diag(RangeLoc, diag::err_for_range_dereference)
|
|
|
|
<< Range->getType() << FixItHint::CreateInsertion(RangeLoc, "*");
|
|
|
|
return SemaRef.ActOnCXXForRangeStmt(ForLoc, LoopVarDecl, ColonLoc,
|
|
|
|
AdjustedRange.get(), RParenLoc,
|
|
|
|
Sema::BFRK_Rebuild);
|
2011-04-15 06:09:26 +08:00
|
|
|
}
|
|
|
|
|
2012-09-21 05:52:32 +08:00
|
|
|
/// BuildCXXForRangeStmt - Build or instantiate a C++11 for-range statement.
|
2011-04-15 06:09:26 +08:00
|
|
|
StmtResult
|
|
|
|
Sema::BuildCXXForRangeStmt(SourceLocation ForLoc, SourceLocation ColonLoc,
|
|
|
|
Stmt *RangeDecl, Stmt *BeginEnd, Expr *Cond,
|
|
|
|
Expr *Inc, Stmt *LoopVarDecl,
|
2012-09-21 05:52:32 +08:00
|
|
|
SourceLocation RParenLoc, BuildForRangeKind Kind) {
|
2011-04-15 06:09:26 +08:00
|
|
|
Scope *S = getCurScope();
|
|
|
|
|
|
|
|
DeclStmt *RangeDS = cast<DeclStmt>(RangeDecl);
|
|
|
|
VarDecl *RangeVar = cast<VarDecl>(RangeDS->getSingleDecl());
|
|
|
|
QualType RangeVarType = RangeVar->getType();
|
|
|
|
|
|
|
|
DeclStmt *LoopVarDS = cast<DeclStmt>(LoopVarDecl);
|
|
|
|
VarDecl *LoopVar = cast<VarDecl>(LoopVarDS->getSingleDecl());
|
|
|
|
|
|
|
|
StmtResult BeginEndDecl = BeginEnd;
|
|
|
|
ExprResult NotEqExpr = Cond, IncrExpr = Inc;
|
|
|
|
|
|
|
|
if (!BeginEndDecl.get() && !RangeVarType->isDependentType()) {
|
|
|
|
SourceLocation RangeLoc = RangeVar->getLocation();
|
|
|
|
|
2011-10-11 06:36:28 +08:00
|
|
|
const QualType RangeVarNonRefType = RangeVarType.getNonReferenceType();
|
|
|
|
|
|
|
|
ExprResult BeginRangeRef = BuildDeclRefExpr(RangeVar, RangeVarNonRefType,
|
|
|
|
VK_LValue, ColonLoc);
|
|
|
|
if (BeginRangeRef.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
ExprResult EndRangeRef = BuildDeclRefExpr(RangeVar, RangeVarNonRefType,
|
|
|
|
VK_LValue, ColonLoc);
|
|
|
|
if (EndRangeRef.isInvalid())
|
2011-04-15 06:09:26 +08:00
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
QualType AutoType = Context.getAutoDeductType();
|
|
|
|
Expr *Range = RangeVar->getInit();
|
|
|
|
if (!Range)
|
|
|
|
return StmtError();
|
|
|
|
QualType RangeType = Range->getType();
|
|
|
|
|
|
|
|
if (RequireCompleteType(RangeLoc, RangeType,
|
2012-05-05 00:32:21 +08:00
|
|
|
diag::err_for_range_incomplete_type))
|
2011-04-15 06:09:26 +08:00
|
|
|
return StmtError();
|
|
|
|
|
|
|
|
// Build auto __begin = begin-expr, __end = end-expr.
|
|
|
|
VarDecl *BeginVar = BuildForRangeVarDecl(*this, ColonLoc, AutoType,
|
|
|
|
"__begin");
|
|
|
|
VarDecl *EndVar = BuildForRangeVarDecl(*this, ColonLoc, AutoType,
|
|
|
|
"__end");
|
|
|
|
|
|
|
|
// Build begin-expr and end-expr and attach to __begin and __end variables.
|
|
|
|
ExprResult BeginExpr, EndExpr;
|
|
|
|
if (const ArrayType *UnqAT = RangeType->getAsArrayTypeUnsafe()) {
|
|
|
|
// - if _RangeT is an array type, begin-expr and end-expr are __range and
|
|
|
|
// __range + __bound, respectively, where __bound is the array bound. If
|
|
|
|
// _RangeT is an array of unknown size or an array of incomplete type,
|
|
|
|
// the program is ill-formed;
|
|
|
|
|
|
|
|
// begin-expr is __range.
|
2011-10-11 06:36:28 +08:00
|
|
|
BeginExpr = BeginRangeRef;
|
|
|
|
if (FinishForRangeVarDecl(*this, BeginVar, BeginRangeRef.get(), ColonLoc,
|
2011-04-15 06:09:26 +08:00
|
|
|
diag::err_for_range_iter_deduction_failure)) {
|
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Find the array bound.
|
|
|
|
ExprResult BoundExpr;
|
|
|
|
if (const ConstantArrayType *CAT = dyn_cast<ConstantArrayType>(UnqAT))
|
|
|
|
BoundExpr = Owned(IntegerLiteral::Create(Context, CAT->getSize(),
|
2011-05-03 07:00:27 +08:00
|
|
|
Context.getPointerDiffType(),
|
|
|
|
RangeLoc));
|
2011-04-15 06:09:26 +08:00
|
|
|
else if (const VariableArrayType *VAT =
|
|
|
|
dyn_cast<VariableArrayType>(UnqAT))
|
2013-04-21 07:28:26 +08:00
|
|
|
// FIXME: Need to build an OpaqueValueExpr for this rather than
|
|
|
|
// recomputing it!
|
2011-04-15 06:09:26 +08:00
|
|
|
BoundExpr = VAT->getSizeExpr();
|
|
|
|
else {
|
|
|
|
// Can't be a DependentSizedArrayType or an IncompleteArrayType since
|
|
|
|
// UnqAT is not incomplete and Range is not type-dependent.
|
2011-09-23 13:06:16 +08:00
|
|
|
llvm_unreachable("Unexpected array type in for-range");
|
2011-04-15 06:09:26 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// end-expr is __range + __bound.
|
2011-10-11 06:36:28 +08:00
|
|
|
EndExpr = ActOnBinOp(S, ColonLoc, tok::plus, EndRangeRef.get(),
|
2011-04-15 06:09:26 +08:00
|
|
|
BoundExpr.get());
|
|
|
|
if (EndExpr.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
if (FinishForRangeVarDecl(*this, EndVar, EndExpr.get(), ColonLoc,
|
|
|
|
diag::err_for_range_iter_deduction_failure)) {
|
|
|
|
NoteForRangeBeginEndFunction(*this, EndExpr.get(), BEF_end);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
} else {
|
2012-08-21 08:52:01 +08:00
|
|
|
OverloadCandidateSet CandidateSet(RangeLoc);
|
|
|
|
Sema::BeginEndFunction BEFFailure;
|
|
|
|
ForRangeStatus RangeStatus =
|
|
|
|
BuildNonArrayForRange(*this, S, BeginRangeRef.get(),
|
|
|
|
EndRangeRef.get(), RangeType,
|
|
|
|
BeginVar, EndVar, ColonLoc, &CandidateSet,
|
|
|
|
&BeginExpr, &EndExpr, &BEFFailure);
|
|
|
|
|
|
|
|
// If building the range failed, try dereferencing the range expression
|
|
|
|
// unless a diagnostic was issued or the end function is problematic.
|
2012-09-21 05:52:32 +08:00
|
|
|
if (Kind == BFRK_Build && RangeStatus == FRS_NoViableFunction &&
|
2012-08-21 08:52:01 +08:00
|
|
|
BEFFailure == BEF_begin) {
|
|
|
|
StmtResult SR = RebuildForRangeWithDereference(*this, S, ForLoc,
|
|
|
|
LoopVarDecl, ColonLoc,
|
|
|
|
Range, RangeLoc,
|
|
|
|
RParenLoc);
|
2012-09-21 05:52:32 +08:00
|
|
|
if (SR.isInvalid() || SR.isUsable())
|
2012-08-21 08:52:01 +08:00
|
|
|
return SR;
|
2011-04-15 06:09:26 +08:00
|
|
|
}
|
|
|
|
|
2012-08-21 08:52:01 +08:00
|
|
|
// Otherwise, emit diagnostics if we haven't already.
|
|
|
|
if (RangeStatus == FRS_NoViableFunction) {
|
2012-09-21 05:52:32 +08:00
|
|
|
Expr *Range = BEFFailure ? EndRangeRef.get() : BeginRangeRef.get();
|
2012-08-21 08:52:01 +08:00
|
|
|
Diag(Range->getLocStart(), diag::err_for_range_invalid)
|
|
|
|
<< RangeLoc << Range->getType() << BEFFailure;
|
2012-12-30 04:03:39 +08:00
|
|
|
CandidateSet.NoteCandidates(*this, OCD_AllCandidates, Range);
|
2012-08-21 08:52:01 +08:00
|
|
|
}
|
|
|
|
// Return an error if no fix was discovered.
|
|
|
|
if (RangeStatus != FRS_Success)
|
2011-04-15 06:09:26 +08:00
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
2012-08-21 08:52:01 +08:00
|
|
|
assert(!BeginExpr.isInvalid() && !EndExpr.isInvalid() &&
|
|
|
|
"invalid range expression in for loop");
|
|
|
|
|
|
|
|
// C++11 [dcl.spec.auto]p7: BeginType and EndType must be the same.
|
2011-04-15 06:09:26 +08:00
|
|
|
QualType BeginType = BeginVar->getType(), EndType = EndVar->getType();
|
|
|
|
if (!Context.hasSameType(BeginType, EndType)) {
|
|
|
|
Diag(RangeLoc, diag::err_for_range_begin_end_types_differ)
|
|
|
|
<< BeginType << EndType;
|
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
NoteForRangeBeginEndFunction(*this, EndExpr.get(), BEF_end);
|
|
|
|
}
|
|
|
|
|
|
|
|
Decl *BeginEndDecls[] = { BeginVar, EndVar };
|
|
|
|
// Claim the type doesn't contain auto: we've already done the checking.
|
|
|
|
DeclGroupPtrTy BeginEndGroup =
|
|
|
|
BuildDeclaratorGroup(BeginEndDecls, 2, /*TypeMayContainAuto=*/false);
|
|
|
|
BeginEndDecl = ActOnDeclStmt(BeginEndGroup, ColonLoc, ColonLoc);
|
|
|
|
|
2011-10-11 06:36:28 +08:00
|
|
|
const QualType BeginRefNonRefType = BeginType.getNonReferenceType();
|
|
|
|
ExprResult BeginRef = BuildDeclRefExpr(BeginVar, BeginRefNonRefType,
|
2011-04-15 06:09:26 +08:00
|
|
|
VK_LValue, ColonLoc);
|
2011-10-11 06:36:28 +08:00
|
|
|
if (BeginRef.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
ExprResult EndRef = BuildDeclRefExpr(EndVar, EndType.getNonReferenceType(),
|
|
|
|
VK_LValue, ColonLoc);
|
2011-10-11 06:36:28 +08:00
|
|
|
if (EndRef.isInvalid())
|
|
|
|
return StmtError();
|
2011-04-15 06:09:26 +08:00
|
|
|
|
|
|
|
// Build and check __begin != __end expression.
|
|
|
|
NotEqExpr = ActOnBinOp(S, ColonLoc, tok::exclaimequal,
|
|
|
|
BeginRef.get(), EndRef.get());
|
|
|
|
NotEqExpr = ActOnBooleanCondition(S, ColonLoc, NotEqExpr.get());
|
|
|
|
NotEqExpr = ActOnFinishFullExpr(NotEqExpr.get());
|
|
|
|
if (NotEqExpr.isInvalid()) {
|
2012-09-07 05:50:08 +08:00
|
|
|
Diag(RangeLoc, diag::note_for_range_invalid_iterator)
|
|
|
|
<< RangeLoc << 0 << BeginRangeRef.get()->getType();
|
2011-04-15 06:09:26 +08:00
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
if (!Context.hasSameType(BeginType, EndType))
|
|
|
|
NoteForRangeBeginEndFunction(*this, EndExpr.get(), BEF_end);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Build and check ++__begin expression.
|
2011-10-11 06:36:28 +08:00
|
|
|
BeginRef = BuildDeclRefExpr(BeginVar, BeginRefNonRefType,
|
|
|
|
VK_LValue, ColonLoc);
|
|
|
|
if (BeginRef.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
IncrExpr = ActOnUnaryOp(S, ColonLoc, tok::plusplus, BeginRef.get());
|
|
|
|
IncrExpr = ActOnFinishFullExpr(IncrExpr.get());
|
|
|
|
if (IncrExpr.isInvalid()) {
|
2012-09-07 05:50:08 +08:00
|
|
|
Diag(RangeLoc, diag::note_for_range_invalid_iterator)
|
|
|
|
<< RangeLoc << 2 << BeginRangeRef.get()->getType() ;
|
2011-04-15 06:09:26 +08:00
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
|
|
|
// Build and check *__begin expression.
|
2011-10-11 06:36:28 +08:00
|
|
|
BeginRef = BuildDeclRefExpr(BeginVar, BeginRefNonRefType,
|
|
|
|
VK_LValue, ColonLoc);
|
|
|
|
if (BeginRef.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
ExprResult DerefExpr = ActOnUnaryOp(S, ColonLoc, tok::star, BeginRef.get());
|
|
|
|
if (DerefExpr.isInvalid()) {
|
2012-09-07 05:50:08 +08:00
|
|
|
Diag(RangeLoc, diag::note_for_range_invalid_iterator)
|
|
|
|
<< RangeLoc << 1 << BeginRangeRef.get()->getType();
|
2011-04-15 06:09:26 +08:00
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
|
2012-09-21 05:52:32 +08:00
|
|
|
// Attach *__begin as initializer for VD. Don't touch it if we're just
|
|
|
|
// trying to determine whether this would be a valid range.
|
|
|
|
if (!LoopVar->isInvalidDecl() && Kind != BFRK_Check) {
|
2011-04-15 06:09:26 +08:00
|
|
|
AddInitializerToDecl(LoopVar, DerefExpr.get(), /*DirectInit=*/false,
|
|
|
|
/*TypeMayContainAuto=*/true);
|
|
|
|
if (LoopVar->isInvalidDecl())
|
|
|
|
NoteForRangeBeginEndFunction(*this, BeginExpr.get(), BEF_begin);
|
|
|
|
}
|
2011-06-22 07:07:19 +08:00
|
|
|
} else {
|
|
|
|
// The range is implicitly used as a placeholder when it is dependent.
|
|
|
|
RangeVar->setUsed();
|
2011-04-15 06:09:26 +08:00
|
|
|
}
|
|
|
|
|
2012-09-21 05:52:32 +08:00
|
|
|
// Don't bother to actually allocate the result if we're just trying to
|
|
|
|
// determine whether it would be valid.
|
|
|
|
if (Kind == BFRK_Check)
|
|
|
|
return StmtResult();
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
return Owned(new (Context) CXXForRangeStmt(RangeDS,
|
|
|
|
cast_or_null<DeclStmt>(BeginEndDecl.get()),
|
|
|
|
NotEqExpr.take(), IncrExpr.take(),
|
|
|
|
LoopVarDS, /*Body=*/0, ForLoc,
|
|
|
|
ColonLoc, RParenLoc));
|
|
|
|
}
|
|
|
|
|
2012-08-11 01:56:09 +08:00
|
|
|
/// FinishObjCForCollectionStmt - Attach the body to a objective-C foreach
|
2012-07-04 06:00:52 +08:00
|
|
|
/// statement.
|
|
|
|
StmtResult Sema::FinishObjCForCollectionStmt(Stmt *S, Stmt *B) {
|
|
|
|
if (!S || !B)
|
|
|
|
return StmtError();
|
|
|
|
ObjCForCollectionStmt * ForStmt = cast<ObjCForCollectionStmt>(S);
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-07-04 06:00:52 +08:00
|
|
|
ForStmt->setBody(B);
|
|
|
|
return S;
|
|
|
|
}
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
/// FinishCXXForRangeStmt - Attach the body to a C++0x for-range statement.
|
|
|
|
/// This is a separate step from ActOnCXXForRangeStmt because analysis of the
|
|
|
|
/// body cannot be performed until after the type of the range variable is
|
|
|
|
/// determined.
|
|
|
|
StmtResult Sema::FinishCXXForRangeStmt(Stmt *S, Stmt *B) {
|
|
|
|
if (!S || !B)
|
|
|
|
return StmtError();
|
|
|
|
|
2012-07-07 03:04:04 +08:00
|
|
|
if (isa<ObjCForCollectionStmt>(S))
|
|
|
|
return FinishObjCForCollectionStmt(S, B);
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-02-15 06:14:32 +08:00
|
|
|
CXXForRangeStmt *ForStmt = cast<CXXForRangeStmt>(S);
|
|
|
|
ForStmt->setBody(B);
|
|
|
|
|
|
|
|
DiagnoseEmptyStmtBody(ForStmt->getRParenLoc(), B,
|
|
|
|
diag::warn_empty_range_based_for_body);
|
|
|
|
|
2011-04-15 06:09:26 +08:00
|
|
|
return S;
|
|
|
|
}
|
|
|
|
|
2011-02-18 04:34:02 +08:00
|
|
|
StmtResult Sema::ActOnGotoStmt(SourceLocation GotoLoc,
|
|
|
|
SourceLocation LabelLoc,
|
|
|
|
LabelDecl *TheDecl) {
|
|
|
|
getCurFunction()->setHasBranchIntoScope();
|
2011-02-17 15:39:24 +08:00
|
|
|
TheDecl->setUsed();
|
|
|
|
return Owned(new (Context) GotoStmt(TheDecl, GotoLoc, LabelLoc));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
2007-05-31 14:00:00 +08:00
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2009-04-19 09:04:21 +08:00
|
|
|
Sema::ActOnIndirectGotoStmt(SourceLocation GotoLoc, SourceLocation StarLoc,
|
2010-08-24 07:25:46 +08:00
|
|
|
Expr *E) {
|
2009-03-26 08:18:06 +08:00
|
|
|
// Convert operand to void*
|
2009-05-16 08:20:29 +08:00
|
|
|
if (!E->isTypeDependent()) {
|
|
|
|
QualType ETy = E->getType();
|
2010-01-31 18:26:25 +08:00
|
|
|
QualType DestTy = Context.getPointerType(Context.VoidTy.withConst());
|
2011-04-09 02:41:53 +08:00
|
|
|
ExprResult ExprRes = Owned(E);
|
2009-05-16 08:20:29 +08:00
|
|
|
AssignConvertType ConvTy =
|
2011-04-09 02:41:53 +08:00
|
|
|
CheckSingleAssignmentConstraints(DestTy, ExprRes);
|
|
|
|
if (ExprRes.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
E = ExprRes.take();
|
2010-01-31 18:26:25 +08:00
|
|
|
if (DiagnoseAssignmentResult(ConvTy, StarLoc, DestTy, ETy, E, AA_Passing))
|
2009-05-16 08:20:29 +08:00
|
|
|
return StmtError();
|
|
|
|
}
|
2010-08-01 08:26:45 +08:00
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
ExprResult ExprRes = ActOnFinishFullExpr(E);
|
|
|
|
if (ExprRes.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
E = ExprRes.take();
|
|
|
|
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->setHasIndirectGoto();
|
2010-08-01 08:26:45 +08:00
|
|
|
|
2009-05-16 08:20:29 +08:00
|
|
|
return Owned(new (Context) IndirectGotoStmt(GotoLoc, StarLoc, E));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2007-09-16 22:56:35 +08:00
|
|
|
Sema::ActOnContinueStmt(SourceLocation ContinueLoc, Scope *CurScope) {
|
2006-11-10 13:17:58 +08:00
|
|
|
Scope *S = CurScope->getContinueParent();
|
|
|
|
if (!S) {
|
|
|
|
// C99 6.8.6.2p1: A break shall appear only in or as a loop body.
|
2009-01-18 21:19:59 +08:00
|
|
|
return StmtError(Diag(ContinueLoc, diag::err_continue_not_in_loop));
|
2006-11-10 13:17:58 +08:00
|
|
|
}
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2009-02-07 09:47:29 +08:00
|
|
|
return Owned(new (Context) ContinueStmt(ContinueLoc));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2007-09-16 22:56:35 +08:00
|
|
|
Sema::ActOnBreakStmt(SourceLocation BreakLoc, Scope *CurScope) {
|
2006-11-10 13:17:58 +08:00
|
|
|
Scope *S = CurScope->getBreakParent();
|
|
|
|
if (!S) {
|
|
|
|
// C99 6.8.6.3p1: A break shall appear only in or as a switch/loop body.
|
2009-01-18 21:19:59 +08:00
|
|
|
return StmtError(Diag(BreakLoc, diag::err_break_not_in_loop_or_switch));
|
2006-11-10 13:17:58 +08:00
|
|
|
}
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2009-02-07 09:47:29 +08:00
|
|
|
return Owned(new (Context) BreakStmt(BreakLoc));
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2011-01-27 15:10:08 +08:00
|
|
|
/// \brief Determine whether the given expression is a candidate for
|
2011-01-22 02:05:27 +08:00
|
|
|
/// copy elision in either a return statement or a throw expression.
|
2010-05-15 14:01:05 +08:00
|
|
|
///
|
2011-01-22 02:05:27 +08:00
|
|
|
/// \param ReturnType If we're determining the copy elision candidate for
|
|
|
|
/// a return statement, this is the return type of the function. If we're
|
|
|
|
/// determining the copy elision candidate for a throw expression, this will
|
|
|
|
/// be a NULL type.
|
2010-05-15 14:01:05 +08:00
|
|
|
///
|
2011-01-22 02:05:27 +08:00
|
|
|
/// \param E The expression being returned from the function or block, or
|
|
|
|
/// being thrown.
|
2010-05-15 14:01:05 +08:00
|
|
|
///
|
2011-05-20 23:00:53 +08:00
|
|
|
/// \param AllowFunctionParameter Whether we allow function parameters to
|
|
|
|
/// be considered NRVO candidates. C++ prohibits this for NRVO itself, but
|
|
|
|
/// we re-use this logic to determine whether we should try to move as part of
|
|
|
|
/// a return or throw (which does allow function parameters).
|
2010-05-15 14:01:05 +08:00
|
|
|
///
|
|
|
|
/// \returns The NRVO candidate variable, if the return statement may use the
|
|
|
|
/// NRVO, or NULL if there is no such candidate.
|
2011-01-22 02:05:27 +08:00
|
|
|
const VarDecl *Sema::getCopyElisionCandidate(QualType ReturnType,
|
|
|
|
Expr *E,
|
|
|
|
bool AllowFunctionParameter) {
|
|
|
|
QualType ExprType = E->getType();
|
2010-05-15 08:13:29 +08:00
|
|
|
// - in a return statement in a function with ...
|
|
|
|
// ... a class return type ...
|
2011-01-22 02:05:27 +08:00
|
|
|
if (!ReturnType.isNull()) {
|
|
|
|
if (!ReturnType->isRecordType())
|
|
|
|
return 0;
|
|
|
|
// ... the same cv-unqualified type as the function return type ...
|
|
|
|
if (!Context.hasSameUnqualifiedType(ReturnType, ExprType))
|
|
|
|
return 0;
|
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
|
|
|
// ... the expression is the name of a non-volatile automatic object
|
2011-01-22 02:05:27 +08:00
|
|
|
// (other than a function or catch-clause parameter)) ...
|
|
|
|
const DeclRefExpr *DR = dyn_cast<DeclRefExpr>(E->IgnoreParens());
|
2012-07-12 06:50:15 +08:00
|
|
|
if (!DR || DR->refersToEnclosingLocal())
|
2010-05-15 14:01:05 +08:00
|
|
|
return 0;
|
2010-05-15 08:13:29 +08:00
|
|
|
const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl());
|
|
|
|
if (!VD)
|
2010-05-15 14:01:05 +08:00
|
|
|
return 0;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-11-11 11:57:31 +08:00
|
|
|
// ...object (other than a function or catch-clause parameter)...
|
|
|
|
if (VD->getKind() != Decl::Var &&
|
|
|
|
!(AllowFunctionParameter && VD->getKind() == Decl::ParmVar))
|
|
|
|
return 0;
|
|
|
|
if (VD->isExceptionVariable()) return 0;
|
|
|
|
|
|
|
|
// ...automatic...
|
|
|
|
if (!VD->hasLocalStorage()) return 0;
|
|
|
|
|
|
|
|
// ...non-volatile...
|
|
|
|
if (VD->getType().isVolatileQualified()) return 0;
|
|
|
|
if (VD->getType()->isReferenceType()) return 0;
|
|
|
|
|
|
|
|
// __block variables can't be allocated in a way that permits NRVO.
|
|
|
|
if (VD->hasAttr<BlocksAttr>()) return 0;
|
|
|
|
|
|
|
|
// Variables with higher required alignment than their type's ABI
|
|
|
|
// alignment cannot use NRVO.
|
|
|
|
if (VD->hasAttr<AlignedAttr>() &&
|
|
|
|
Context.getDeclAlign(VD) > Context.getTypeAlignInChars(VD->getType()))
|
|
|
|
return 0;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-11-11 11:57:31 +08:00
|
|
|
return VD;
|
2010-05-15 08:13:29 +08:00
|
|
|
}
|
|
|
|
|
2011-01-22 05:08:57 +08:00
|
|
|
/// \brief Perform the initialization of a potentially-movable value, which
|
|
|
|
/// is the result of return value.
|
2011-01-22 03:38:21 +08:00
|
|
|
///
|
|
|
|
/// This routine implements C++0x [class.copy]p33, which attempts to treat
|
|
|
|
/// returned lvalues as rvalues in certain cases (to prefer move construction),
|
|
|
|
/// then falls back to treating them as lvalues if that failed.
|
2011-01-27 15:10:08 +08:00
|
|
|
ExprResult
|
2011-01-22 05:08:57 +08:00
|
|
|
Sema::PerformMoveOrCopyInitialization(const InitializedEntity &Entity,
|
|
|
|
const VarDecl *NRVOCandidate,
|
|
|
|
QualType ResultType,
|
2011-07-07 06:04:06 +08:00
|
|
|
Expr *Value,
|
|
|
|
bool AllowNRVO) {
|
2011-01-22 03:38:21 +08:00
|
|
|
// C++0x [class.copy]p33:
|
2011-01-27 15:10:08 +08:00
|
|
|
// When the criteria for elision of a copy operation are met or would
|
|
|
|
// be met save for the fact that the source object is a function
|
|
|
|
// parameter, and the object to be copied is designated by an lvalue,
|
2011-01-22 03:38:21 +08:00
|
|
|
// overload resolution to select the constructor for the copy is first
|
|
|
|
// performed as if the object were designated by an rvalue.
|
|
|
|
ExprResult Res = ExprError();
|
2011-07-07 06:04:06 +08:00
|
|
|
if (AllowNRVO &&
|
|
|
|
(NRVOCandidate || getCopyElisionCandidate(ResultType, Value, true))) {
|
2011-01-27 15:10:08 +08:00
|
|
|
ImplicitCastExpr AsRvalue(ImplicitCastExpr::OnStack,
|
2012-05-15 13:04:02 +08:00
|
|
|
Value->getType(), CK_NoOp, Value, VK_XValue);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
Expr *InitExpr = &AsRvalue;
|
2011-01-27 15:10:08 +08:00
|
|
|
InitializationKind Kind
|
2011-01-22 05:08:57 +08:00
|
|
|
= InitializationKind::CreateCopy(Value->getLocStart(),
|
|
|
|
Value->getLocStart());
|
|
|
|
InitializationSequence Seq(*this, Entity, Kind, &InitExpr, 1);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
|
|
|
// [...] If overload resolution fails, or if the type of the first
|
2011-01-22 03:38:21 +08:00
|
|
|
// parameter of the selected constructor is not an rvalue reference
|
2011-01-27 15:09:49 +08:00
|
|
|
// to the object's type (possibly cv-qualified), overload resolution
|
2011-01-22 03:38:21 +08:00
|
|
|
// is performed again, considering the object as an lvalue.
|
2011-06-05 20:23:28 +08:00
|
|
|
if (Seq) {
|
2011-01-22 03:38:21 +08:00
|
|
|
for (InitializationSequence::step_iterator Step = Seq.step_begin(),
|
|
|
|
StepEnd = Seq.step_end();
|
|
|
|
Step != StepEnd; ++Step) {
|
2011-06-05 20:23:28 +08:00
|
|
|
if (Step->Kind != InitializationSequence::SK_ConstructorInitialization)
|
2011-01-22 03:38:21 +08:00
|
|
|
continue;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
|
|
|
CXXConstructorDecl *Constructor
|
2011-01-22 03:38:21 +08:00
|
|
|
= cast<CXXConstructorDecl>(Step->Function.Function);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
const RValueReferenceType *RRefType
|
2011-01-22 05:08:57 +08:00
|
|
|
= Constructor->getParamDecl(0)->getType()
|
|
|
|
->getAs<RValueReferenceType>();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
// If we don't meet the criteria, break out now.
|
2011-01-27 15:10:08 +08:00
|
|
|
if (!RRefType ||
|
2011-01-22 05:08:57 +08:00
|
|
|
!Context.hasSameUnqualifiedType(RRefType->getPointeeType(),
|
|
|
|
Context.getTypeDeclType(Constructor->getParent())))
|
2011-01-22 03:38:21 +08:00
|
|
|
break;
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
// Promote "AsRvalue" to the heap, since we now need this
|
|
|
|
// expression node to persist.
|
2011-01-22 05:08:57 +08:00
|
|
|
Value = ImplicitCastExpr::Create(Context, Value->getType(),
|
2012-05-15 13:04:02 +08:00
|
|
|
CK_NoOp, Value, 0, VK_XValue);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
// Complete type-checking the initialization of the return type
|
|
|
|
// using the constructor we found.
|
2011-01-22 05:08:57 +08:00
|
|
|
Res = Seq.Perform(*this, Entity, Kind, MultiExprArg(&Value, 1));
|
2011-01-22 03:38:21 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
// Either we didn't meet the criteria for treating an lvalue as an rvalue,
|
2011-01-27 15:10:08 +08:00
|
|
|
// above, or overload resolution failed. Either way, we need to try
|
2011-01-22 03:38:21 +08:00
|
|
|
// (again) now with the return value expression as written.
|
|
|
|
if (Res.isInvalid())
|
2011-01-22 05:08:57 +08:00
|
|
|
Res = PerformCopyInitialization(Entity, SourceLocation(), Value);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-01-22 03:38:21 +08:00
|
|
|
return Res;
|
|
|
|
}
|
|
|
|
|
2012-01-26 11:00:14 +08:00
|
|
|
/// ActOnCapScopeReturnStmt - Utility routine to type-check return statements
|
|
|
|
/// for capturing scopes.
|
2008-09-04 02:15:37 +08:00
|
|
|
///
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2012-01-26 11:00:14 +08:00
|
|
|
Sema::ActOnCapScopeReturnStmt(SourceLocation ReturnLoc, Expr *RetValExp) {
|
|
|
|
// If this is the first return we've seen, infer the return type.
|
|
|
|
// [expr.prim.lambda]p4 in C++11; block literals follow a superset of those
|
|
|
|
// rules which allows multiple return statements.
|
|
|
|
CapturingScopeInfo *CurCap = cast<CapturingScopeInfo>(getCurFunction());
|
2012-07-03 05:19:23 +08:00
|
|
|
QualType FnRetType = CurCap->ReturnType;
|
|
|
|
|
|
|
|
// For blocks/lambdas with implicit return types, we check each return
|
|
|
|
// statement individually, and deduce the common return type when the block
|
|
|
|
// or lambda is completed.
|
2012-01-26 11:00:14 +08:00
|
|
|
if (CurCap->HasImplicitReturnType) {
|
2012-02-10 02:40:39 +08:00
|
|
|
if (RetValExp && !isa<InitListExpr>(RetValExp)) {
|
2011-04-09 02:41:53 +08:00
|
|
|
ExprResult Result = DefaultFunctionArrayLvalueConversion(RetValExp);
|
|
|
|
if (Result.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
RetValExp = Result.take();
|
2011-06-05 13:04:23 +08:00
|
|
|
|
2012-07-03 05:19:23 +08:00
|
|
|
if (!RetValExp->isTypeDependent())
|
|
|
|
FnRetType = RetValExp->getType();
|
|
|
|
else
|
|
|
|
FnRetType = CurCap->ReturnType = Context.DependentTy;
|
2012-08-11 01:56:09 +08:00
|
|
|
} else {
|
2012-02-10 02:40:39 +08:00
|
|
|
if (RetValExp) {
|
|
|
|
// C++11 [expr.lambda.prim]p4 bans inferring the result from an
|
|
|
|
// initializer list, because it is not an expression (even
|
|
|
|
// though we represent it as one). We still deduce 'void'.
|
|
|
|
Diag(ReturnLoc, diag::err_lambda_return_init_list)
|
|
|
|
<< RetValExp->getSourceRange();
|
|
|
|
}
|
|
|
|
|
2012-07-03 05:19:23 +08:00
|
|
|
FnRetType = Context.VoidTy;
|
2011-12-04 07:53:56 +08:00
|
|
|
}
|
2012-07-03 05:19:23 +08:00
|
|
|
|
|
|
|
// Although we'll properly infer the type of the block once it's completed,
|
|
|
|
// make sure we provide a return type now for better error recovery.
|
|
|
|
if (CurCap->ReturnType.isNull())
|
|
|
|
CurCap->ReturnType = FnRetType;
|
2008-09-04 02:15:37 +08:00
|
|
|
}
|
2012-01-26 11:00:14 +08:00
|
|
|
assert(!FnRetType.isNull());
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2012-02-16 00:20:15 +08:00
|
|
|
if (BlockScopeInfo *CurBlock = dyn_cast<BlockScopeInfo>(CurCap)) {
|
2012-01-26 11:00:14 +08:00
|
|
|
if (CurBlock->FunctionType->getAs<FunctionType>()->getNoReturnAttr()) {
|
|
|
|
Diag(ReturnLoc, diag::err_noreturn_block_has_return_expr);
|
|
|
|
return StmtError();
|
|
|
|
}
|
2013-04-17 03:37:38 +08:00
|
|
|
} else if (CapturedRegionScopeInfo *CurRegion =
|
|
|
|
dyn_cast<CapturedRegionScopeInfo>(CurCap)) {
|
|
|
|
Diag(ReturnLoc, diag::err_return_in_captured_stmt) << CurRegion->getRegionName();
|
|
|
|
return StmtError();
|
2012-02-16 00:20:15 +08:00
|
|
|
} else {
|
|
|
|
LambdaScopeInfo *LSI = cast<LambdaScopeInfo>(CurCap);
|
|
|
|
if (LSI->CallOperator->getType()->getAs<FunctionType>()->getNoReturnAttr()){
|
|
|
|
Diag(ReturnLoc, diag::err_noreturn_lambda_has_return_expr);
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
}
|
2009-04-30 05:40:37 +08:00
|
|
|
|
2008-09-04 02:15:37 +08:00
|
|
|
// Otherwise, verify that this result type matches the previous one. We are
|
|
|
|
// pickier with blocks than for normal functions because we don't have GCC
|
|
|
|
// compatibility to worry about here.
|
2011-08-18 05:34:14 +08:00
|
|
|
const VarDecl *NRVOCandidate = 0;
|
2011-08-18 06:09:46 +08:00
|
|
|
if (FnRetType->isDependentType()) {
|
|
|
|
// Delay processing for now. TODO: there are lots of dependent
|
|
|
|
// types we can conclusively prove aren't void.
|
|
|
|
} else if (FnRetType->isVoidType()) {
|
2012-02-23 01:38:04 +08:00
|
|
|
if (RetValExp && !isa<InitListExpr>(RetValExp) &&
|
2012-03-11 15:00:24 +08:00
|
|
|
!(getLangOpts().CPlusPlus &&
|
2011-08-18 06:09:46 +08:00
|
|
|
(RetValExp->isTypeDependent() ||
|
|
|
|
RetValExp->getType()->isVoidType()))) {
|
2012-03-22 00:45:13 +08:00
|
|
|
if (!getLangOpts().CPlusPlus &&
|
|
|
|
RetValExp->getType()->isVoidType())
|
2012-03-22 04:28:39 +08:00
|
|
|
Diag(ReturnLoc, diag::ext_return_has_void_expr) << "literal" << 2;
|
2012-03-22 00:45:13 +08:00
|
|
|
else {
|
|
|
|
Diag(ReturnLoc, diag::err_return_block_has_expr);
|
|
|
|
RetValExp = 0;
|
|
|
|
}
|
2008-09-04 02:15:37 +08:00
|
|
|
}
|
2010-05-15 14:01:05 +08:00
|
|
|
} else if (!RetValExp) {
|
2011-08-18 06:09:46 +08:00
|
|
|
return StmtError(Diag(ReturnLoc, diag::err_block_return_missing_expr));
|
|
|
|
} else if (!RetValExp->isTypeDependent()) {
|
|
|
|
// we have a non-void block with an expression, continue checking
|
|
|
|
|
|
|
|
// C99 6.8.6.4p3(136): The return statement is not an assignment. The
|
|
|
|
// overlap restriction of subclause 6.5.16.1 does not apply to the case of
|
|
|
|
// function return.
|
|
|
|
|
|
|
|
// In C++ the return statement is handled via a copy initialization.
|
|
|
|
// the C version of which boils down to CheckSingleAssignmentConstraints.
|
|
|
|
NRVOCandidate = getCopyElisionCandidate(FnRetType, RetValExp, false);
|
|
|
|
InitializedEntity Entity = InitializedEntity::InitializeResult(ReturnLoc,
|
|
|
|
FnRetType,
|
2011-12-04 01:47:53 +08:00
|
|
|
NRVOCandidate != 0);
|
2011-08-18 06:09:46 +08:00
|
|
|
ExprResult Res = PerformMoveOrCopyInitialization(Entity, NRVOCandidate,
|
|
|
|
FnRetType, RetValExp);
|
|
|
|
if (Res.isInvalid()) {
|
|
|
|
// FIXME: Cleanup temporaries here, anyway?
|
|
|
|
return StmtError();
|
2010-01-30 02:30:20 +08:00
|
|
|
}
|
2011-08-18 06:09:46 +08:00
|
|
|
RetValExp = Res.take();
|
|
|
|
CheckReturnStackAddr(RetValExp, FnRetType, ReturnLoc);
|
2011-08-18 05:34:14 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2011-08-18 05:34:14 +08:00
|
|
|
if (RetValExp) {
|
2013-01-15 06:39:08 +08:00
|
|
|
ExprResult ER = ActOnFinishFullExpr(RetValExp, ReturnLoc);
|
|
|
|
if (ER.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
RetValExp = ER.take();
|
2009-02-05 06:31:32 +08:00
|
|
|
}
|
2011-08-18 06:09:46 +08:00
|
|
|
ReturnStmt *Result = new (Context) ReturnStmt(ReturnLoc, RetValExp,
|
|
|
|
NRVOCandidate);
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2012-07-03 05:19:23 +08:00
|
|
|
// If we need to check for the named return value optimization,
|
|
|
|
// or if we need to infer the return type,
|
|
|
|
// save the return statement in our scope for later processing.
|
|
|
|
if (CurCap->HasImplicitReturnType ||
|
|
|
|
(getLangOpts().CPlusPlus && FnRetType->isRecordType() &&
|
|
|
|
!CurContext->isDependentContext()))
|
2010-05-15 14:01:05 +08:00
|
|
|
FunctionScopes.back()->Returns.push_back(Result);
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-05-15 14:01:05 +08:00
|
|
|
return Owned(Result);
|
2008-09-04 02:15:37 +08:00
|
|
|
}
|
2006-11-10 13:07:45 +08:00
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnReturnStmt(SourceLocation ReturnLoc, Expr *RetValExp) {
|
2011-05-20 23:32:55 +08:00
|
|
|
// Check for unexpanded parameter packs.
|
|
|
|
if (RetValExp && DiagnoseUnexpandedParameterPack(RetValExp))
|
|
|
|
return StmtError();
|
2012-08-11 01:56:09 +08:00
|
|
|
|
2012-01-26 11:00:14 +08:00
|
|
|
if (isa<CapturingScopeInfo>(getCurFunction()))
|
|
|
|
return ActOnCapScopeReturnStmt(ReturnLoc, RetValExp);
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2008-12-05 07:50:19 +08:00
|
|
|
QualType FnRetType;
|
2012-03-30 09:13:43 +08:00
|
|
|
QualType RelatedRetType;
|
2009-04-29 08:43:21 +08:00
|
|
|
if (const FunctionDecl *FD = getCurFunctionDecl()) {
|
2008-12-05 07:50:19 +08:00
|
|
|
FnRetType = FD->getResultType();
|
2013-01-17 09:30:42 +08:00
|
|
|
if (FD->isNoReturn())
|
2009-06-01 03:32:13 +08:00
|
|
|
Diag(ReturnLoc, diag::warn_noreturn_function_has_return_expr)
|
2012-01-05 08:49:17 +08:00
|
|
|
<< FD->getDeclName();
|
2011-06-11 09:09:30 +08:00
|
|
|
} else if (ObjCMethodDecl *MD = getCurMethodDecl()) {
|
2012-03-30 09:13:43 +08:00
|
|
|
FnRetType = MD->getResultType();
|
2011-06-11 09:09:30 +08:00
|
|
|
if (MD->hasRelatedResultType() && MD->getClassInterface()) {
|
|
|
|
// In the implementation of a method with a related return type, the
|
2012-08-11 01:56:09 +08:00
|
|
|
// type used to type-check the validity of return statements within the
|
2011-06-11 09:09:30 +08:00
|
|
|
// method body is a pointer to the type of the class being implemented.
|
2012-03-30 09:13:43 +08:00
|
|
|
RelatedRetType = Context.getObjCInterfaceType(MD->getClassInterface());
|
|
|
|
RelatedRetType = Context.getObjCObjectPointerType(RelatedRetType);
|
2011-06-11 09:09:30 +08:00
|
|
|
}
|
|
|
|
} else // If we don't have a function/method context, bail.
|
2009-03-03 08:45:38 +08:00
|
|
|
return StmtError();
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2010-05-15 14:01:05 +08:00
|
|
|
ReturnStmt *Result = 0;
|
2008-01-05 02:04:52 +08:00
|
|
|
if (FnRetType->isVoidType()) {
|
2011-06-01 15:44:31 +08:00
|
|
|
if (RetValExp) {
|
2012-02-22 18:50:08 +08:00
|
|
|
if (isa<InitListExpr>(RetValExp)) {
|
|
|
|
// We simply never allow init lists as the return value of void
|
|
|
|
// functions. This is compatible because this was never allowed before,
|
|
|
|
// so there's no legacy code to deal with.
|
|
|
|
NamedDecl *CurDecl = getCurFunctionOrMethodDecl();
|
|
|
|
int FunctionKind = 0;
|
|
|
|
if (isa<ObjCMethodDecl>(CurDecl))
|
|
|
|
FunctionKind = 1;
|
|
|
|
else if (isa<CXXConstructorDecl>(CurDecl))
|
|
|
|
FunctionKind = 2;
|
|
|
|
else if (isa<CXXDestructorDecl>(CurDecl))
|
|
|
|
FunctionKind = 3;
|
|
|
|
|
|
|
|
Diag(ReturnLoc, diag::err_return_init_list)
|
|
|
|
<< CurDecl->getDeclName() << FunctionKind
|
|
|
|
<< RetValExp->getSourceRange();
|
|
|
|
|
|
|
|
// Drop the expression.
|
|
|
|
RetValExp = 0;
|
|
|
|
} else if (!RetValExp->isTypeDependent()) {
|
2011-06-01 15:44:31 +08:00
|
|
|
// C99 6.8.6.4p1 (ext_ since GCC warns)
|
|
|
|
unsigned D = diag::ext_return_has_expr;
|
|
|
|
if (RetValExp->getType()->isVoidType())
|
|
|
|
D = diag::ext_return_has_void_expr;
|
|
|
|
else {
|
|
|
|
ExprResult Result = Owned(RetValExp);
|
|
|
|
Result = IgnoredValueConversions(Result.take());
|
|
|
|
if (Result.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
RetValExp = Result.take();
|
|
|
|
RetValExp = ImpCastExprToType(RetValExp,
|
|
|
|
Context.VoidTy, CK_ToVoid).take();
|
|
|
|
}
|
2009-01-18 21:19:59 +08:00
|
|
|
|
2011-06-01 15:44:31 +08:00
|
|
|
// return (some void expression); is legal in C++.
|
|
|
|
if (D != diag::ext_return_has_void_expr ||
|
2012-03-11 15:00:24 +08:00
|
|
|
!getLangOpts().CPlusPlus) {
|
2011-06-01 15:44:31 +08:00
|
|
|
NamedDecl *CurDecl = getCurFunctionOrMethodDecl();
|
2011-06-30 16:56:22 +08:00
|
|
|
|
|
|
|
int FunctionKind = 0;
|
|
|
|
if (isa<ObjCMethodDecl>(CurDecl))
|
|
|
|
FunctionKind = 1;
|
|
|
|
else if (isa<CXXConstructorDecl>(CurDecl))
|
|
|
|
FunctionKind = 2;
|
|
|
|
else if (isa<CXXDestructorDecl>(CurDecl))
|
|
|
|
FunctionKind = 3;
|
|
|
|
|
2011-06-01 15:44:31 +08:00
|
|
|
Diag(ReturnLoc, D)
|
2011-06-30 16:56:22 +08:00
|
|
|
<< CurDecl->getDeclName() << FunctionKind
|
2011-06-01 15:44:31 +08:00
|
|
|
<< RetValExp->getSourceRange();
|
|
|
|
}
|
2008-12-18 10:03:48 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2012-02-22 18:50:08 +08:00
|
|
|
if (RetValExp) {
|
2013-01-15 06:39:08 +08:00
|
|
|
ExprResult ER = ActOnFinishFullExpr(RetValExp, ReturnLoc);
|
|
|
|
if (ER.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
RetValExp = ER.take();
|
2012-02-22 18:50:08 +08:00
|
|
|
}
|
2007-05-29 22:23:36 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-05-15 14:01:05 +08:00
|
|
|
Result = new (Context) ReturnStmt(ReturnLoc, RetValExp, 0);
|
|
|
|
} else if (!RetValExp && !FnRetType->isDependentType()) {
|
2008-11-19 16:23:25 +08:00
|
|
|
unsigned DiagID = diag::warn_return_missing_expr; // C90 6.6.6.4p4
|
|
|
|
// C99 6.8.6.4p1 (ext_ since GCC warns)
|
2012-03-11 15:00:24 +08:00
|
|
|
if (getLangOpts().C99) DiagID = diag::ext_return_missing_expr;
|
2008-11-19 16:23:25 +08:00
|
|
|
|
|
|
|
if (FunctionDecl *FD = getCurFunctionDecl())
|
2008-11-24 05:45:46 +08:00
|
|
|
Diag(ReturnLoc, DiagID) << FD->getIdentifier() << 0/*fn*/;
|
2008-11-19 16:23:25 +08:00
|
|
|
else
|
2008-11-24 05:45:46 +08:00
|
|
|
Diag(ReturnLoc, DiagID) << getCurMethodDecl()->getDeclName() << 1/*meth*/;
|
2010-05-15 14:01:05 +08:00
|
|
|
Result = new (Context) ReturnStmt(ReturnLoc);
|
|
|
|
} else {
|
2013-03-05 09:27:54 +08:00
|
|
|
assert(RetValExp || FnRetType->isDependentType());
|
2010-05-15 14:01:05 +08:00
|
|
|
const VarDecl *NRVOCandidate = 0;
|
|
|
|
if (!FnRetType->isDependentType() && !RetValExp->isTypeDependent()) {
|
|
|
|
// we have a non-void function with an expression, continue checking
|
|
|
|
|
2013-03-19 15:04:25 +08:00
|
|
|
QualType RetType = (RelatedRetType.isNull() ? FnRetType : RelatedRetType);
|
2012-03-30 09:13:43 +08:00
|
|
|
|
2010-05-15 14:01:05 +08:00
|
|
|
// C99 6.8.6.4p3(136): The return statement is not an assignment. The
|
|
|
|
// overlap restriction of subclause 6.5.16.1 does not apply to the case of
|
|
|
|
// function return.
|
|
|
|
|
2011-06-17 07:24:51 +08:00
|
|
|
// In C++ the return statement is handled via a copy initialization,
|
2010-05-15 14:01:05 +08:00
|
|
|
// the C version of which boils down to CheckSingleAssignmentConstraints.
|
2011-01-22 02:05:27 +08:00
|
|
|
NRVOCandidate = getCopyElisionCandidate(FnRetType, RetValExp, false);
|
2011-01-27 15:10:08 +08:00
|
|
|
InitializedEntity Entity = InitializedEntity::InitializeResult(ReturnLoc,
|
2013-03-19 15:04:25 +08:00
|
|
|
RetType,
|
2011-06-02 08:47:27 +08:00
|
|
|
NRVOCandidate != 0);
|
2011-01-27 15:10:08 +08:00
|
|
|
ExprResult Res = PerformMoveOrCopyInitialization(Entity, NRVOCandidate,
|
2013-03-19 15:04:25 +08:00
|
|
|
RetType, RetValExp);
|
2010-05-15 14:01:05 +08:00
|
|
|
if (Res.isInvalid()) {
|
2013-03-19 15:04:25 +08:00
|
|
|
// FIXME: Clean up temporaries here anyway?
|
2010-05-15 14:01:05 +08:00
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
RetValExp = Res.takeAs<Expr>();
|
2013-03-19 15:04:25 +08:00
|
|
|
|
|
|
|
// If we have a related result type, we need to implicitly
|
|
|
|
// convert back to the formal result type. We can't pretend to
|
|
|
|
// initialize the result again --- we might end double-retaining
|
|
|
|
// --- so instead we initialize a notional temporary; this can
|
|
|
|
// lead to less-than-great diagnostics, but this stage is much
|
|
|
|
// less likely to fail than the previous stage.
|
|
|
|
if (!RelatedRetType.isNull()) {
|
|
|
|
Entity = InitializedEntity::InitializeTemporary(FnRetType);
|
|
|
|
Res = PerformCopyInitialization(Entity, ReturnLoc, RetValExp);
|
|
|
|
if (Res.isInvalid()) {
|
|
|
|
// FIXME: Clean up temporaries here anyway?
|
|
|
|
return StmtError();
|
|
|
|
}
|
|
|
|
RetValExp = Res.takeAs<Expr>();
|
|
|
|
}
|
|
|
|
|
|
|
|
CheckReturnStackAddr(RetValExp, FnRetType, ReturnLoc);
|
2009-11-14 09:20:54 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-10-08 10:01:28 +08:00
|
|
|
if (RetValExp) {
|
2013-01-15 06:39:08 +08:00
|
|
|
ExprResult ER = ActOnFinishFullExpr(RetValExp, ReturnLoc);
|
|
|
|
if (ER.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
RetValExp = ER.take();
|
2010-10-08 10:01:28 +08:00
|
|
|
}
|
2010-05-15 14:01:05 +08:00
|
|
|
Result = new (Context) ReturnStmt(ReturnLoc, RetValExp, NRVOCandidate);
|
2008-12-06 07:32:09 +08:00
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
|
|
|
// If we need to check for the named return value optimization, save the
|
2010-05-15 14:01:05 +08:00
|
|
|
// return statement in our scope for later processing.
|
2012-03-11 15:00:24 +08:00
|
|
|
if (getLangOpts().CPlusPlus && FnRetType->isRecordType() &&
|
2010-05-15 14:01:05 +08:00
|
|
|
!CurContext->isDependentContext())
|
|
|
|
FunctionScopes.back()->Returns.push_back(Result);
|
2012-06-21 02:51:04 +08:00
|
|
|
|
2010-05-15 14:01:05 +08:00
|
|
|
return Owned(Result);
|
2006-11-10 13:07:45 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2009-01-19 01:43:11 +08:00
|
|
|
Sema::ActOnObjCAtCatchStmt(SourceLocation AtLoc,
|
2010-08-21 17:40:31 +08:00
|
|
|
SourceLocation RParen, Decl *Parm,
|
2010-08-24 07:25:46 +08:00
|
|
|
Stmt *Body) {
|
2010-08-21 17:40:31 +08:00
|
|
|
VarDecl *Var = cast_or_null<VarDecl>(Parm);
|
2010-04-27 01:32:49 +08:00
|
|
|
if (Var && Var->isInvalidDecl())
|
|
|
|
return StmtError();
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(new (Context) ObjCAtCatchStmt(AtLoc, RParen, Var, Body));
|
2007-11-02 07:59:59 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnObjCAtFinallyStmt(SourceLocation AtLoc, Stmt *Body) {
|
|
|
|
return Owned(new (Context) ObjCAtFinallyStmt(AtLoc, Body));
|
2007-11-02 08:18:53 +08:00
|
|
|
}
|
2007-11-02 23:39:31 +08:00
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2011-01-27 15:10:08 +08:00
|
|
|
Sema::ActOnObjCAtTryStmt(SourceLocation AtLoc, Stmt *Try,
|
2010-08-24 07:25:46 +08:00
|
|
|
MultiStmtArg CatchStmts, Stmt *Finally) {
|
2012-03-11 15:00:24 +08:00
|
|
|
if (!getLangOpts().ObjCExceptions)
|
2011-02-20 07:53:54 +08:00
|
|
|
Diag(AtLoc, diag::err_objc_exceptions_disabled) << "@try";
|
|
|
|
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->setHasBranchProtectedScope();
|
2010-04-24 06:50:49 +08:00
|
|
|
unsigned NumCatchStmts = CatchStmts.size();
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(ObjCAtTryStmt::Create(Context, AtLoc, Try,
|
2012-08-24 07:38:35 +08:00
|
|
|
CatchStmts.data(),
|
2010-04-24 06:50:49 +08:00
|
|
|
NumCatchStmts,
|
2010-08-24 07:25:46 +08:00
|
|
|
Finally));
|
2007-11-02 23:39:31 +08:00
|
|
|
}
|
|
|
|
|
2012-05-09 05:41:25 +08:00
|
|
|
StmtResult Sema::BuildObjCAtThrowStmt(SourceLocation AtLoc, Expr *Throw) {
|
2010-04-23 05:44:01 +08:00
|
|
|
if (Throw) {
|
2011-04-09 02:41:53 +08:00
|
|
|
ExprResult Result = DefaultLvalueConversion(Throw);
|
|
|
|
if (Result.isInvalid())
|
|
|
|
return StmtError();
|
2010-12-15 12:42:30 +08:00
|
|
|
|
2013-01-15 06:39:08 +08:00
|
|
|
Result = ActOnFinishFullExpr(Result.take());
|
|
|
|
if (Result.isInvalid())
|
|
|
|
return StmtError();
|
|
|
|
Throw = Result.take();
|
|
|
|
|
2010-04-23 05:44:01 +08:00
|
|
|
QualType ThrowType = Throw->getType();
|
|
|
|
// Make sure the expression type is an ObjC pointer or "void *".
|
|
|
|
if (!ThrowType->isDependentType() &&
|
|
|
|
!ThrowType->isObjCObjectPointerType()) {
|
|
|
|
const PointerType *PT = ThrowType->getAs<PointerType>();
|
|
|
|
if (!PT || !PT->getPointeeType()->isVoidType())
|
|
|
|
return StmtError(Diag(AtLoc, diag::error_objc_throw_expects_object)
|
|
|
|
<< Throw->getType() << Throw->getSourceRange());
|
|
|
|
}
|
|
|
|
}
|
2011-01-27 15:10:08 +08:00
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(new (Context) ObjCAtThrowStmt(AtLoc, Throw));
|
2010-04-23 05:44:01 +08:00
|
|
|
}
|
|
|
|
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2011-01-27 15:10:08 +08:00
|
|
|
Sema::ActOnObjCAtThrowStmt(SourceLocation AtLoc, Expr *Throw,
|
2010-04-23 05:44:01 +08:00
|
|
|
Scope *CurScope) {
|
2012-03-11 15:00:24 +08:00
|
|
|
if (!getLangOpts().ObjCExceptions)
|
2011-02-20 07:53:54 +08:00
|
|
|
Diag(AtLoc, diag::err_objc_exceptions_disabled) << "@throw";
|
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
if (!Throw) {
|
2009-02-12 04:05:44 +08:00
|
|
|
// @throw without an expression designates a rethrow (which much occur
|
|
|
|
// in the context of an @catch clause).
|
|
|
|
Scope *AtCatchParent = CurScope;
|
|
|
|
while (AtCatchParent && !AtCatchParent->isAtCatchScope())
|
|
|
|
AtCatchParent = AtCatchParent->getParent();
|
|
|
|
if (!AtCatchParent)
|
2009-02-13 02:09:32 +08:00
|
|
|
return StmtError(Diag(AtLoc, diag::error_rethrow_used_outside_catch));
|
2011-01-27 15:10:08 +08:00
|
|
|
}
|
2010-08-24 07:25:46 +08:00
|
|
|
return BuildObjCAtThrowStmt(AtLoc, Throw);
|
2007-11-07 10:00:49 +08:00
|
|
|
}
|
2007-11-02 23:39:31 +08:00
|
|
|
|
2011-07-28 05:50:02 +08:00
|
|
|
ExprResult
|
|
|
|
Sema::ActOnObjCAtSynchronizedOperand(SourceLocation atLoc, Expr *operand) {
|
|
|
|
ExprResult result = DefaultLvalueConversion(operand);
|
|
|
|
if (result.isInvalid())
|
|
|
|
return ExprError();
|
|
|
|
operand = result.take();
|
2010-12-15 12:42:30 +08:00
|
|
|
|
2009-04-21 14:11:25 +08:00
|
|
|
// Make sure the expression type is an ObjC pointer or "void *".
|
2011-07-28 05:50:02 +08:00
|
|
|
QualType type = operand->getType();
|
|
|
|
if (!type->isDependentType() &&
|
|
|
|
!type->isObjCObjectPointerType()) {
|
|
|
|
const PointerType *pointerType = type->getAs<PointerType>();
|
|
|
|
if (!pointerType || !pointerType->getPointeeType()->isVoidType())
|
|
|
|
return Diag(atLoc, diag::error_objc_synchronized_expects_object)
|
|
|
|
<< type << operand->getSourceRange();
|
2009-04-21 14:11:25 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2011-07-28 05:50:02 +08:00
|
|
|
// The operand to @synchronized is a full-expression.
|
2013-01-15 06:39:08 +08:00
|
|
|
return ActOnFinishFullExpr(operand);
|
2011-07-28 05:50:02 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
StmtResult
|
|
|
|
Sema::ActOnObjCAtSynchronizedStmt(SourceLocation AtLoc, Expr *SyncExpr,
|
|
|
|
Stmt *SyncBody) {
|
|
|
|
// We can't jump into or indirect-jump out of a @synchronized block.
|
|
|
|
getCurFunction()->setHasBranchProtectedScope();
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(new (Context) ObjCAtSynchronizedStmt(AtLoc, SyncExpr, SyncBody));
|
2008-01-30 03:14:59 +08:00
|
|
|
}
|
2008-12-23 03:15:10 +08:00
|
|
|
|
|
|
|
/// ActOnCXXCatchBlock - Takes an exception declaration and a handler block
|
|
|
|
/// and creates a proper catch handler from them.
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-21 17:40:31 +08:00
|
|
|
Sema::ActOnCXXCatchBlock(SourceLocation CatchLoc, Decl *ExDecl,
|
2010-08-24 07:25:46 +08:00
|
|
|
Stmt *HandlerBlock) {
|
2008-12-23 03:15:10 +08:00
|
|
|
// There's nothing to test that ActOnExceptionDecl didn't already test.
|
2009-02-07 09:47:29 +08:00
|
|
|
return Owned(new (Context) CXXCatchStmt(CatchLoc,
|
2010-08-21 17:40:31 +08:00
|
|
|
cast_or_null<VarDecl>(ExDecl),
|
2010-08-24 07:25:46 +08:00
|
|
|
HandlerBlock));
|
2008-12-23 03:15:10 +08:00
|
|
|
}
|
2008-12-23 05:35:02 +08:00
|
|
|
|
2011-06-16 07:02:42 +08:00
|
|
|
StmtResult
|
|
|
|
Sema::ActOnObjCAutoreleasePoolStmt(SourceLocation AtLoc, Stmt *Body) {
|
|
|
|
getCurFunction()->setHasBranchProtectedScope();
|
|
|
|
return Owned(new (Context) ObjCAutoreleasePoolStmt(AtLoc, Body));
|
|
|
|
}
|
|
|
|
|
2010-07-27 05:25:24 +08:00
|
|
|
namespace {
|
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
class TypeWithHandler {
|
|
|
|
QualType t;
|
|
|
|
CXXCatchStmt *stmt;
|
|
|
|
public:
|
|
|
|
TypeWithHandler(const QualType &type, CXXCatchStmt *statement)
|
|
|
|
: t(type), stmt(statement) {}
|
|
|
|
|
2009-09-25 03:53:00 +08:00
|
|
|
// An arbitrary order is fine as long as it places identical
|
|
|
|
// types next to each other.
|
2009-07-30 01:15:45 +08:00
|
|
|
bool operator<(const TypeWithHandler &y) const {
|
2009-09-25 03:53:00 +08:00
|
|
|
if (t.getAsOpaquePtr() < y.t.getAsOpaquePtr())
|
2009-07-30 01:15:45 +08:00
|
|
|
return true;
|
2009-09-25 03:53:00 +08:00
|
|
|
if (t.getAsOpaquePtr() > y.t.getAsOpaquePtr())
|
2009-07-30 01:15:45 +08:00
|
|
|
return false;
|
|
|
|
else
|
|
|
|
return getTypeSpecStartLoc() < y.getTypeSpecStartLoc();
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
bool operator==(const TypeWithHandler& other) const {
|
2009-09-25 03:53:00 +08:00
|
|
|
return t == other.t;
|
2009-07-30 01:15:45 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
CXXCatchStmt *getCatchStmt() const { return stmt; }
|
|
|
|
SourceLocation getTypeSpecStartLoc() const {
|
|
|
|
return stmt->getExceptionDecl()->getTypeSpecStartLoc();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
2010-07-27 05:25:24 +08:00
|
|
|
}
|
|
|
|
|
2008-12-23 05:35:02 +08:00
|
|
|
/// ActOnCXXTryBlock - Takes a try compound-statement and a number of
|
|
|
|
/// handlers and creates a try statement from them.
|
2010-08-24 14:29:42 +08:00
|
|
|
StmtResult
|
2010-08-24 07:25:46 +08:00
|
|
|
Sema::ActOnCXXTryBlock(SourceLocation TryLoc, Stmt *TryBlock,
|
2008-12-23 05:35:02 +08:00
|
|
|
MultiStmtArg RawHandlers) {
|
2011-02-23 11:46:46 +08:00
|
|
|
// Don't report an error if 'try' is used in system headers.
|
2012-03-11 15:00:24 +08:00
|
|
|
if (!getLangOpts().CXXExceptions &&
|
2011-02-23 11:46:46 +08:00
|
|
|
!getSourceManager().isInSystemHeader(TryLoc))
|
|
|
|
Diag(TryLoc, diag::err_exceptions_disabled) << "try";
|
2011-02-20 03:26:44 +08:00
|
|
|
|
2008-12-23 05:35:02 +08:00
|
|
|
unsigned NumHandlers = RawHandlers.size();
|
|
|
|
assert(NumHandlers > 0 &&
|
|
|
|
"The parser shouldn't call this if there are no handlers.");
|
2012-08-24 07:38:35 +08:00
|
|
|
Stmt **Handlers = RawHandlers.data();
|
2008-12-23 05:35:02 +08:00
|
|
|
|
2011-07-23 18:55:15 +08:00
|
|
|
SmallVector<TypeWithHandler, 8> TypesWithHandlers;
|
2009-09-09 23:08:12 +08:00
|
|
|
|
|
|
|
for (unsigned i = 0; i < NumHandlers; ++i) {
|
2011-07-23 18:55:15 +08:00
|
|
|
CXXCatchStmt *Handler = cast<CXXCatchStmt>(Handlers[i]);
|
2009-07-30 01:15:45 +08:00
|
|
|
if (!Handler->getExceptionDecl()) {
|
|
|
|
if (i < NumHandlers - 1)
|
|
|
|
return StmtError(Diag(Handler->getLocStart(),
|
|
|
|
diag::err_early_catch_all));
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
continue;
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
const QualType CaughtType = Handler->getCaughtType();
|
|
|
|
const QualType CanonicalCaughtType = Context.getCanonicalType(CaughtType);
|
|
|
|
TypesWithHandlers.push_back(TypeWithHandler(CanonicalCaughtType, Handler));
|
|
|
|
}
|
|
|
|
|
|
|
|
// Detect handlers for the same type as an earlier one.
|
|
|
|
if (NumHandlers > 1) {
|
|
|
|
llvm::array_pod_sort(TypesWithHandlers.begin(), TypesWithHandlers.end());
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
TypeWithHandler prev = TypesWithHandlers[0];
|
|
|
|
for (unsigned i = 1; i < TypesWithHandlers.size(); ++i) {
|
|
|
|
TypeWithHandler curr = TypesWithHandlers[i];
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
if (curr == prev) {
|
|
|
|
Diag(curr.getTypeSpecStartLoc(),
|
|
|
|
diag::warn_exception_caught_by_earlier_handler)
|
|
|
|
<< curr.getCatchStmt()->getCaughtType().getAsString();
|
|
|
|
Diag(prev.getTypeSpecStartLoc(),
|
|
|
|
diag::note_previous_exception_handler)
|
|
|
|
<< prev.getCatchStmt()->getCaughtType().getAsString();
|
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2009-07-30 01:15:45 +08:00
|
|
|
prev = curr;
|
|
|
|
}
|
2008-12-23 05:35:02 +08:00
|
|
|
}
|
2009-09-09 23:08:12 +08:00
|
|
|
|
2010-08-25 16:40:02 +08:00
|
|
|
getCurFunction()->setHasBranchProtectedScope();
|
2010-08-01 08:26:45 +08:00
|
|
|
|
2008-12-23 05:35:02 +08:00
|
|
|
// FIXME: We should detect handlers that cannot catch anything because an
|
|
|
|
// earlier handler catches a superclass. Need to find a method that is not
|
|
|
|
// quadratic for this.
|
|
|
|
// Neither of these are explicitly forbidden, but every compiler detects them
|
|
|
|
// and warns.
|
|
|
|
|
2010-08-24 07:25:46 +08:00
|
|
|
return Owned(CXXTryStmt::Create(Context, TryLoc, TryBlock,
|
2012-12-30 04:13:03 +08:00
|
|
|
llvm::makeArrayRef(Handlers, NumHandlers)));
|
2008-12-23 05:35:02 +08:00
|
|
|
}
|
2011-04-28 09:08:34 +08:00
|
|
|
|
|
|
|
StmtResult
|
|
|
|
Sema::ActOnSEHTryBlock(bool IsCXXTry,
|
|
|
|
SourceLocation TryLoc,
|
|
|
|
Stmt *TryBlock,
|
|
|
|
Stmt *Handler) {
|
|
|
|
assert(TryBlock && Handler);
|
|
|
|
|
|
|
|
getCurFunction()->setHasBranchProtectedScope();
|
|
|
|
|
|
|
|
return Owned(SEHTryStmt::Create(Context,IsCXXTry,TryLoc,TryBlock,Handler));
|
|
|
|
}
|
|
|
|
|
|
|
|
StmtResult
|
|
|
|
Sema::ActOnSEHExceptBlock(SourceLocation Loc,
|
|
|
|
Expr *FilterExpr,
|
|
|
|
Stmt *Block) {
|
|
|
|
assert(FilterExpr && Block);
|
|
|
|
|
|
|
|
if(!FilterExpr->getType()->isIntegerType()) {
|
2011-06-02 08:47:27 +08:00
|
|
|
return StmtError(Diag(FilterExpr->getExprLoc(),
|
|
|
|
diag::err_filter_expression_integral)
|
|
|
|
<< FilterExpr->getType());
|
2011-04-28 09:08:34 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return Owned(SEHExceptStmt::Create(Context,Loc,FilterExpr,Block));
|
|
|
|
}
|
|
|
|
|
|
|
|
StmtResult
|
|
|
|
Sema::ActOnSEHFinallyBlock(SourceLocation Loc,
|
|
|
|
Stmt *Block) {
|
|
|
|
assert(Block);
|
|
|
|
return Owned(SEHFinallyStmt::Create(Context,Loc,Block));
|
|
|
|
}
|
2011-10-25 09:33:02 +08:00
|
|
|
|
|
|
|
StmtResult Sema::BuildMSDependentExistsStmt(SourceLocation KeywordLoc,
|
|
|
|
bool IsIfExists,
|
|
|
|
NestedNameSpecifierLoc QualifierLoc,
|
|
|
|
DeclarationNameInfo NameInfo,
|
|
|
|
Stmt *Nested)
|
|
|
|
{
|
|
|
|
return new (Context) MSDependentExistsStmt(KeywordLoc, IsIfExists,
|
2012-08-11 01:56:09 +08:00
|
|
|
QualifierLoc, NameInfo,
|
2011-10-25 09:33:02 +08:00
|
|
|
cast<CompoundStmt>(Nested));
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2012-08-11 01:56:09 +08:00
|
|
|
StmtResult Sema::ActOnMSDependentExistsStmt(SourceLocation KeywordLoc,
|
2011-10-25 09:33:02 +08:00
|
|
|
bool IsIfExists,
|
2012-08-11 01:56:09 +08:00
|
|
|
CXXScopeSpec &SS,
|
2011-10-25 09:33:02 +08:00
|
|
|
UnqualifiedId &Name,
|
|
|
|
Stmt *Nested) {
|
2012-08-11 01:56:09 +08:00
|
|
|
return BuildMSDependentExistsStmt(KeywordLoc, IsIfExists,
|
2011-10-25 09:33:02 +08:00
|
|
|
SS.getWithLocInContext(Context),
|
|
|
|
GetNameFromUnqualifiedId(Name),
|
|
|
|
Nested);
|
|
|
|
}
|
2013-04-17 03:37:38 +08:00
|
|
|
|
|
|
|
RecordDecl*
|
|
|
|
Sema::CreateCapturedStmtRecordDecl(CapturedDecl *&CD, SourceLocation Loc)
|
|
|
|
{
|
|
|
|
DeclContext *DC = CurContext;
|
|
|
|
while (!(DC->isFunctionOrMethod() || DC->isRecord() || DC->isFileContext()))
|
|
|
|
DC = DC->getParent();
|
|
|
|
|
|
|
|
RecordDecl *RD = 0;
|
|
|
|
if (getLangOpts().CPlusPlus)
|
|
|
|
RD = CXXRecordDecl::Create(Context, TTK_Struct, DC, Loc, Loc, /*Id=*/0);
|
|
|
|
else
|
|
|
|
RD = RecordDecl::Create(Context, TTK_Struct, DC, Loc, Loc, /*Id=*/0);
|
|
|
|
|
|
|
|
DC->addDecl(RD);
|
|
|
|
RD->setImplicit();
|
|
|
|
RD->startDefinition();
|
|
|
|
|
|
|
|
CD = CapturedDecl::Create(Context, CurContext);
|
|
|
|
DC->addDecl(CD);
|
|
|
|
|
|
|
|
return RD;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void buildCapturedStmtCaptureList(
|
|
|
|
SmallVectorImpl<CapturedStmt::Capture> &Captures,
|
|
|
|
SmallVectorImpl<Expr *> &CaptureInits,
|
|
|
|
ArrayRef<CapturingScopeInfo::Capture> Candidates) {
|
|
|
|
|
|
|
|
typedef ArrayRef<CapturingScopeInfo::Capture>::const_iterator CaptureIter;
|
|
|
|
for (CaptureIter Cap = Candidates.begin(); Cap != Candidates.end(); ++Cap) {
|
|
|
|
|
|
|
|
if (Cap->isThisCapture()) {
|
|
|
|
Captures.push_back(CapturedStmt::Capture(Cap->getLocation(),
|
|
|
|
CapturedStmt::VCK_This));
|
|
|
|
CaptureInits.push_back(Cap->getCopyExpr());
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
assert(Cap->isReferenceCapture() &&
|
|
|
|
"non-reference capture not yet implemented");
|
|
|
|
|
|
|
|
Captures.push_back(CapturedStmt::Capture(Cap->getLocation(),
|
|
|
|
CapturedStmt::VCK_ByRef,
|
|
|
|
Cap->getVariable()));
|
|
|
|
CaptureInits.push_back(Cap->getCopyExpr());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
void Sema::ActOnCapturedRegionStart(SourceLocation Loc, Scope *CurScope,
|
|
|
|
CapturedRegionScopeInfo::CapturedRegionKind Kind) {
|
|
|
|
CapturedDecl *CD = 0;
|
|
|
|
RecordDecl *RD = CreateCapturedStmtRecordDecl(CD, Loc);
|
|
|
|
|
|
|
|
// Enter the capturing scope for this captured region.
|
|
|
|
PushCapturedRegionScope(CurScope, CD, RD, Kind);
|
|
|
|
|
|
|
|
if (CurScope)
|
|
|
|
PushDeclContext(CurScope, CD);
|
|
|
|
else
|
|
|
|
CurContext = CD;
|
|
|
|
|
|
|
|
PushExpressionEvaluationContext(PotentiallyEvaluated);
|
|
|
|
}
|
|
|
|
|
|
|
|
void Sema::ActOnCapturedRegionError(bool IsInstantiation) {
|
|
|
|
DiscardCleanupsInEvaluationContext();
|
|
|
|
PopExpressionEvaluationContext();
|
|
|
|
|
|
|
|
if (!IsInstantiation)
|
|
|
|
PopDeclContext();
|
|
|
|
|
|
|
|
CapturedRegionScopeInfo *RSI = getCurCapturedRegion();
|
|
|
|
RecordDecl *Record = RSI->TheRecordDecl;
|
|
|
|
Record->setInvalidDecl();
|
|
|
|
|
|
|
|
SmallVector<Decl*, 4> Fields;
|
|
|
|
for (RecordDecl::field_iterator I = Record->field_begin(),
|
|
|
|
E = Record->field_end(); I != E; ++I)
|
|
|
|
Fields.push_back(*I);
|
|
|
|
ActOnFields(/*Scope=*/0, Record->getLocation(), Record, Fields,
|
|
|
|
SourceLocation(), SourceLocation(), /*AttributeList=*/0);
|
|
|
|
|
|
|
|
PopFunctionScopeInfo();
|
|
|
|
}
|
|
|
|
|
|
|
|
StmtResult Sema::ActOnCapturedRegionEnd(Stmt *S) {
|
|
|
|
CapturedRegionScopeInfo *RSI = getCurCapturedRegion();
|
|
|
|
|
|
|
|
SmallVector<CapturedStmt::Capture, 4> Captures;
|
|
|
|
SmallVector<Expr *, 4> CaptureInits;
|
|
|
|
buildCapturedStmtCaptureList(Captures, CaptureInits, RSI->Captures);
|
|
|
|
|
|
|
|
CapturedDecl *CD = RSI->TheCapturedDecl;
|
|
|
|
RecordDecl *RD = RSI->TheRecordDecl;
|
|
|
|
|
|
|
|
CapturedStmt *Res = CapturedStmt::Create(getASTContext(), S, Captures,
|
|
|
|
CaptureInits, CD, RD);
|
|
|
|
|
|
|
|
CD->setBody(Res->getCapturedStmt());
|
|
|
|
RD->completeDefinition();
|
|
|
|
|
|
|
|
PopDeclContext();
|
|
|
|
PopFunctionScopeInfo();
|
|
|
|
|
|
|
|
return Owned(Res);
|
|
|
|
}
|