diff options
| author | Douglas Gregor <dgregor@apple.com> | 2009-10-01 23:25:31 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2009-10-01 23:25:31 +0000 |
| commit | 1be8aee8745e8b814ad2f151aa214b0ef07833db (patch) | |
| tree | 2f43d734a9059a88a89ccd6e05ad80cf0ffc6568 /lib/Sema/SemaStmt.cpp | |
| parent | b289b3f324eb10d416b87080e39b315f6c17a695 (diff) | |
When the return type of a function is dependent, don't perform any
of the flow-control checks for falling off the end of a function,
since the return type may instantiate to void. Similarly, if a
return statement has an expression and the return type of the function
is void, don't complain if the expression is type-dependent, since
that type could instantiate to void.
Fixes PR5071.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@83222 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaStmt.cpp')
| -rw-r--r-- | lib/Sema/SemaStmt.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Sema/SemaStmt.cpp b/lib/Sema/SemaStmt.cpp index a73c261d97..5dd56b2221 100644 --- a/lib/Sema/SemaStmt.cpp +++ b/lib/Sema/SemaStmt.cpp @@ -892,7 +892,8 @@ Sema::ActOnReturnStmt(SourceLocation ReturnLoc, ExprArg rex) { return StmtError(); if (FnRetType->isVoidType()) { - if (RetValExp) {// C99 6.8.6.4p1 (ext_ since GCC warns) + if (RetValExp && !RetValExp->isTypeDependent()) { + // 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; |