blob: 3bc1b9cd54e5670d6cd6592573cdc55e6ceefa12 [file] [log] [blame]
Chris Lattneraf8d5812006-11-10 05:07:45 +00001//===--- SemaStmt.cpp - Semantic Analysis for Statements ------------------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file was developed by Chris Lattner and is distributed under
6// the University of Illinois Open Source License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file implements semantic analysis for statements.
11//
12//===----------------------------------------------------------------------===//
13
14#include "Sema.h"
Chris Lattnerfc1c44a2007-08-23 05:46:52 +000015#include "clang/AST/ASTContext.h"
Steve Naroff8eeeb132007-05-08 21:09:37 +000016#include "clang/AST/Expr.h"
Chris Lattnerfc1c44a2007-08-23 05:46:52 +000017#include "clang/AST/Stmt.h"
Chris Lattnereaafe1222006-11-10 05:17:58 +000018#include "clang/Parse/Scope.h"
19#include "clang/Basic/Diagnostic.h"
Steve Naroff6f49f5d2007-05-29 14:23:36 +000020#include "clang/Basic/LangOptions.h"
21#include "clang/Lex/IdentifierTable.h"
Chris Lattneraf8d5812006-11-10 05:07:45 +000022using namespace clang;
23
Chris Lattner1ec5f562007-06-27 05:38:08 +000024Sema::StmtResult Sema::ParseExprStmt(ExprTy *expr) {
25 Expr *E = static_cast<Expr*>(expr);
Steve Narofff8fd09e2007-07-27 22:15:19 +000026 assert(E && "ParseExprStmt(): missing expression");
Chris Lattner1ec5f562007-06-27 05:38:08 +000027
28 // Exprs are statements, so there is no need to do a conversion here. However,
29 // diagnose some potentially bad code.
30 if (!E->hasLocalSideEffect())
Chris Lattner9b3b9a12007-06-27 06:08:24 +000031 Diag(E->getExprLoc(), diag::warn_unused_expr, E->getSourceRange());
Chris Lattner1ec5f562007-06-27 05:38:08 +000032
33 return E;
34}
35
36
Chris Lattner0f203a72007-05-28 01:45:28 +000037Sema::StmtResult Sema::ParseNullStmt(SourceLocation SemiLoc) {
38 return new NullStmt(SemiLoc);
39}
40
Steve Naroff2a8ad182007-05-29 22:59:26 +000041Sema::StmtResult Sema::ParseDeclStmt(DeclTy *decl) {
Steve Naroff56faab22007-05-30 04:20:12 +000042 if (decl)
43 return new DeclStmt(static_cast<Decl *>(decl));
44 else
45 return true; // error
Steve Naroff2a8ad182007-05-29 22:59:26 +000046}
Chris Lattneraf8d5812006-11-10 05:07:45 +000047
48Action::StmtResult
49Sema::ParseCompoundStmt(SourceLocation L, SourceLocation R,
50 StmtTy **Elts, unsigned NumElts) {
Chris Lattner3bbe7be2007-05-20 22:47:04 +000051 return new CompoundStmt((Stmt**)Elts, NumElts);
Chris Lattneraf8d5812006-11-10 05:07:45 +000052}
53
54Action::StmtResult
Chris Lattner0c46c5d2007-06-02 23:53:17 +000055Sema::ParseCaseStmt(SourceLocation CaseLoc, ExprTy *lhsval,
Chris Lattner46eeb222007-07-18 02:28:47 +000056 SourceLocation DotDotDotLoc, ExprTy *rhsval,
Chris Lattner39407372007-07-21 03:00:26 +000057 SourceLocation ColonLoc, StmtTy *subStmt) {
58 Stmt *SubStmt = static_cast<Stmt*>(subStmt);
Chris Lattner54f4d2b2007-07-23 17:05:23 +000059 Expr *LHSVal = ((Expr *)lhsval), *RHSVal = ((Expr *)rhsval);
Steve Naroff8eeeb132007-05-08 21:09:37 +000060 assert((LHSVal != 0) && "missing expression in case statement");
Anders Carlsson51873c22007-07-22 07:07:56 +000061
Chris Lattner0c46c5d2007-06-02 23:53:17 +000062 SourceLocation ExpLoc;
Steve Naroff8eeeb132007-05-08 21:09:37 +000063 // C99 6.8.4.2p3: The expression shall be an integer constant.
Chris Lattner39407372007-07-21 03:00:26 +000064 if (!LHSVal->isIntegerConstantExpr(Context, &ExpLoc)) {
65 Diag(ExpLoc, diag::err_case_label_not_integer_constant_expr,
66 LHSVal->getSourceRange());
67 return SubStmt;
68 }
Steve Naroff8eeeb132007-05-08 21:09:37 +000069
Chris Lattner46eeb222007-07-18 02:28:47 +000070 // GCC extension: The expression shall be an integer constant.
Chris Lattner39407372007-07-21 03:00:26 +000071 if (RHSVal && !RHSVal->isIntegerConstantExpr(Context, &ExpLoc)) {
72 Diag(ExpLoc, diag::err_case_label_not_integer_constant_expr,
73 RHSVal->getSourceRange());
Chris Lattnerfc1c44a2007-08-23 05:46:52 +000074 RHSVal = 0; // Recover by just forgetting about it.
Chris Lattner46eeb222007-07-18 02:28:47 +000075 }
Chris Lattner54f4d2b2007-07-23 17:05:23 +000076
77 if (SwitchStack.empty()) {
78 Diag(CaseLoc, diag::err_case_not_in_switch);
79 return SubStmt;
80 }
Chris Lattner35e287b2007-06-03 01:44:43 +000081
Anders Carlsson51873c22007-07-22 07:07:56 +000082 CaseStmt *CS = new CaseStmt(LHSVal, RHSVal, SubStmt);
Chris Lattner54f4d2b2007-07-23 17:05:23 +000083 SwitchStack.back()->addSwitchCase(CS);
Anders Carlsson51873c22007-07-22 07:07:56 +000084 return CS;
Chris Lattneraf8d5812006-11-10 05:07:45 +000085}
86
87Action::StmtResult
Chris Lattner46eeb222007-07-18 02:28:47 +000088Sema::ParseDefaultStmt(SourceLocation DefaultLoc, SourceLocation ColonLoc,
Chris Lattner39407372007-07-21 03:00:26 +000089 StmtTy *subStmt, Scope *CurScope) {
90 Stmt *SubStmt = static_cast<Stmt*>(subStmt);
Chris Lattner46eeb222007-07-18 02:28:47 +000091
Chris Lattner54f4d2b2007-07-23 17:05:23 +000092 if (SwitchStack.empty()) {
Chris Lattner39407372007-07-21 03:00:26 +000093 Diag(DefaultLoc, diag::err_default_not_in_switch);
94 return SubStmt;
95 }
96
Chris Lattner39407372007-07-21 03:00:26 +000097 DefaultStmt *DS = new DefaultStmt(DefaultLoc, SubStmt);
Chris Lattner54f4d2b2007-07-23 17:05:23 +000098 SwitchStack.back()->addSwitchCase(DS);
Anders Carlsson51873c22007-07-22 07:07:56 +000099
Chris Lattner46eeb222007-07-18 02:28:47 +0000100 return DS;
Chris Lattneraf8d5812006-11-10 05:07:45 +0000101}
102
103Action::StmtResult
104Sema::ParseLabelStmt(SourceLocation IdentLoc, IdentifierInfo *II,
Chris Lattner39407372007-07-21 03:00:26 +0000105 SourceLocation ColonLoc, StmtTy *subStmt) {
106 Stmt *SubStmt = static_cast<Stmt*>(subStmt);
Chris Lattnere2473062007-05-28 06:28:18 +0000107 // Look up the record for this label identifier.
108 LabelStmt *&LabelDecl = LabelMap[II];
109
110 // If not forward referenced or defined already, just create a new LabelStmt.
111 if (LabelDecl == 0)
Chris Lattner39407372007-07-21 03:00:26 +0000112 return LabelDecl = new LabelStmt(IdentLoc, II, SubStmt);
Chris Lattnere2473062007-05-28 06:28:18 +0000113
Chris Lattnereefa10e2007-05-28 06:56:27 +0000114 assert(LabelDecl->getID() == II && "Label mismatch!");
Chris Lattnere2473062007-05-28 06:28:18 +0000115
116 // Otherwise, this label was either forward reference or multiply defined. If
117 // multiply defined, reject it now.
118 if (LabelDecl->getSubStmt()) {
Chris Lattnereefa10e2007-05-28 06:56:27 +0000119 Diag(IdentLoc, diag::err_redefinition_of_label, LabelDecl->getName());
Chris Lattnere2473062007-05-28 06:28:18 +0000120 Diag(LabelDecl->getIdentLoc(), diag::err_previous_definition);
Chris Lattner39407372007-07-21 03:00:26 +0000121 return SubStmt;
Chris Lattnere2473062007-05-28 06:28:18 +0000122 }
123
124 // Otherwise, this label was forward declared, and we just found its real
125 // definition. Fill in the forward definition and return it.
126 LabelDecl->setIdentLoc(IdentLoc);
Chris Lattner39407372007-07-21 03:00:26 +0000127 LabelDecl->setSubStmt(SubStmt);
Chris Lattnere2473062007-05-28 06:28:18 +0000128 return LabelDecl;
Chris Lattneraf8d5812006-11-10 05:07:45 +0000129}
130
131Action::StmtResult
132Sema::ParseIfStmt(SourceLocation IfLoc, ExprTy *CondVal,
133 StmtTy *ThenVal, SourceLocation ElseLoc,
134 StmtTy *ElseVal) {
Steve Naroff86272ea2007-05-29 02:14:17 +0000135 Expr *condExpr = (Expr *)CondVal;
136 assert(condExpr && "ParseIfStmt(): missing expression");
137
Steve Naroff31090012007-07-16 21:54:35 +0000138 DefaultFunctionArrayConversion(condExpr);
139 QualType condType = condExpr->getType();
Steve Naroff86272ea2007-05-29 02:14:17 +0000140
141 if (!condType->isScalarType()) // C99 6.8.4.1p1
142 return Diag(IfLoc, diag::err_typecheck_statement_requires_scalar,
143 condType.getAsString(), condExpr->getSourceRange());
144
145 return new IfStmt(condExpr, (Stmt*)ThenVal, (Stmt*)ElseVal);
Chris Lattneraf8d5812006-11-10 05:07:45 +0000146}
Steve Naroff86272ea2007-05-29 02:14:17 +0000147
Chris Lattneraf8d5812006-11-10 05:07:45 +0000148Action::StmtResult
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000149Sema::StartSwitchStmt(ExprTy *cond) {
150 Expr *Cond = static_cast<Expr*>(cond);
151
152 // C99 6.8.4.2p5 - Integer promotions are performed on the controlling expr.
153 UsualUnaryConversions(Cond);
154
155 SwitchStmt *SS = new SwitchStmt(Cond);
Anders Carlsson51873c22007-07-22 07:07:56 +0000156 SwitchStack.push_back(SS);
157 return SS;
158}
Chris Lattner46eeb222007-07-18 02:28:47 +0000159
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000160/// ConvertIntegerToTypeWarnOnOverflow - Convert the specified APInt to have
161/// the specified width and sign. If an overflow occurs, detect it and emit
162/// the specified diagnostic.
163void Sema::ConvertIntegerToTypeWarnOnOverflow(llvm::APSInt &Val,
164 unsigned NewWidth, bool NewSign,
165 SourceLocation Loc,
166 unsigned DiagID) {
167 // Perform a conversion to the promoted condition type if needed.
168 if (NewWidth > Val.getBitWidth()) {
169 // If this is an extension, just do it.
170 llvm::APSInt OldVal(Val);
171 Val.extend(NewWidth);
172
173 // If the input was signed and negative and the output is unsigned,
174 // warn.
175 if (!NewSign && OldVal.isSigned() && OldVal.isNegative())
176 Diag(Loc, DiagID, OldVal.toString(), Val.toString());
177
178 Val.setIsSigned(NewSign);
179 } else if (NewWidth < Val.getBitWidth()) {
180 // If this is a truncation, check for overflow.
181 llvm::APSInt ConvVal(Val);
182 ConvVal.trunc(NewWidth);
183 ConvVal.extend(Val.getBitWidth());
184 if (ConvVal != Val)
185 Diag(Loc, DiagID, Val.toString(), ConvVal.toString());
186
187 // Regardless of whether a diagnostic was emitted, really do the
188 // truncation.
189 Val.trunc(NewWidth);
190 } else if (NewSign != Val.isSigned()) {
191 // Convert the sign to match the sign of the condition. This can cause
192 // overflow as well: unsigned(INTMIN)
193 llvm::APSInt OldVal(Val);
194 Val.setIsSigned(NewSign);
195
196 if (Val.isNegative()) // Sign bit changes meaning.
197 Diag(Loc, DiagID, OldVal.toString(), Val.toString());
198 }
199}
200
Anders Carlsson51873c22007-07-22 07:07:56 +0000201Action::StmtResult
202Sema::FinishSwitchStmt(SourceLocation SwitchLoc, StmtTy *Switch, ExprTy *Body) {
203 Stmt *BodyStmt = (Stmt*)Body;
204
205 SwitchStmt *SS = SwitchStack.back();
206 assert(SS == (SwitchStmt*)Switch && "switch stack missing push/pop!");
207
208 SS->setBody(BodyStmt);
209 SwitchStack.pop_back();
210
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000211 Expr *CondExpr = SS->getCond();
212 QualType CondType = CondExpr->getType();
Chris Lattner46eeb222007-07-18 02:28:47 +0000213
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000214 if (!CondType->isIntegerType()) { // C99 6.8.4.2p1
Anders Carlsson51873c22007-07-22 07:07:56 +0000215 Diag(SwitchLoc, diag::err_typecheck_statement_requires_integer,
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000216 CondType.getAsString(), CondExpr->getSourceRange());
217 return true;
Anders Carlsson51873c22007-07-22 07:07:56 +0000218 }
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000219
220 // Get the bitwidth of the switched-on value before promotions. We must
221 // convert the integer case values to this width before comparison.
222 unsigned CondWidth = Context.getTypeSize(CondType, SwitchLoc);
223 bool CondIsSigned = CondType->isSignedIntegerType();
224
225 // Accumulate all of the case values in a vector so that we can sort them
226 // and detect duplicates. This vector contains the APInt for the case after
227 // it has been converted to the condition type.
228 llvm::SmallVector<std::pair<llvm::APSInt, CaseStmt*>, 64> CaseVals;
229
230 // Keep track of any GNU case ranges we see. The APSInt is the low value.
231 std::vector<std::pair<llvm::APSInt, CaseStmt*> > CaseRanges;
232
233 DefaultStmt *TheDefaultStmt = 0;
Anders Carlsson51873c22007-07-22 07:07:56 +0000234
235 // FIXME: The list of cases is backwards and needs to be reversed.
236 for (SwitchCase *SC = SS->getSwitchCaseList(); SC;
237 SC = SC->getNextSwitchCase()) {
238 if (DefaultStmt *DS = dyn_cast<DefaultStmt>(SC)) {
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000239 if (TheDefaultStmt) {
240 Diag(DS->getDefaultLoc(), diag::err_multiple_default_labels_defined);
241 Diag(TheDefaultStmt->getDefaultLoc(), diag::err_first_label);
Anders Carlsson51873c22007-07-22 07:07:56 +0000242
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000243 // FIXME: Remove the default statement from the switch block so that
244 // we'll return a valid AST. This requires recursing down the
245 // AST and finding it, not something we are set up to do right now. For
246 // now, just lop the entire switch stmt out of the AST.
247 return true;
Anders Carlsson51873c22007-07-22 07:07:56 +0000248 }
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000249 TheDefaultStmt = DS;
Anders Carlsson51873c22007-07-22 07:07:56 +0000250
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000251 } else {
252 CaseStmt *CS = cast<CaseStmt>(SC);
253
254 // We already verified that the expression has a i-c-e value (C99
255 // 6.8.4.2p3) - get that value now.
256 llvm::APSInt LoVal(32);
257 CS->getLHS()->isIntegerConstantExpr(LoVal, Context);
258
259 // Convert the value to the same width/sign as the condition.
260 ConvertIntegerToTypeWarnOnOverflow(LoVal, CondWidth, CondIsSigned,
261 CS->getLHS()->getLocStart(),
262 diag::warn_case_value_overflow);
Anders Carlsson51873c22007-07-22 07:07:56 +0000263
Chris Lattnerfc1c44a2007-08-23 05:46:52 +0000264 // Remember the converted value.
265 CaseVals.push_back(std::make_pair(LoVal, CS));
266
267 // Just remember where the case range was.
268 if (CS->getRHS())
269 CaseRanges.push_back(std::make_pair(LoVal, CS));
270 }
271 }
272
273
274
275#if 0
276 assert(CaseRanges.empty() && "FIXME: Check case ranges for overlap etc");
277 // TODO: if the low value is bigger than the high value, the case is
278 // empty: emit "empty range specified" warning and drop the c
279
280 llvm::APSInt HiVal(32);
281 RHS->isIntegerConstantExpr(HiVal, Context);
282
283 // Convert the value to the same width/sign as the condition.
284 ConvertIntegerToTypeWarnOnOverflow(HiVal, CondWidth, CondIsSigned);
285
286 // If low value and high value equal, just forget about it as a range
287 // for the purposes of checking: it identifies a single value.
288 if (LoVal == HiVal)
289 continue;
290
291#endif
292
Anders Carlsson51873c22007-07-22 07:07:56 +0000293 return SS;
Chris Lattneraf8d5812006-11-10 05:07:45 +0000294}
295
296Action::StmtResult
Steve Naroff86272ea2007-05-29 02:14:17 +0000297Sema::ParseWhileStmt(SourceLocation WhileLoc, ExprTy *Cond, StmtTy *Body) {
298 Expr *condExpr = (Expr *)Cond;
299 assert(condExpr && "ParseWhileStmt(): missing expression");
300
Steve Naroff31090012007-07-16 21:54:35 +0000301 DefaultFunctionArrayConversion(condExpr);
302 QualType condType = condExpr->getType();
Steve Naroff86272ea2007-05-29 02:14:17 +0000303
304 if (!condType->isScalarType()) // C99 6.8.5p2
305 return Diag(WhileLoc, diag::err_typecheck_statement_requires_scalar,
306 condType.getAsString(), condExpr->getSourceRange());
307
308 return new WhileStmt(condExpr, (Stmt*)Body);
Chris Lattneraf8d5812006-11-10 05:07:45 +0000309}
310
311Action::StmtResult
312Sema::ParseDoStmt(SourceLocation DoLoc, StmtTy *Body,
313 SourceLocation WhileLoc, ExprTy *Cond) {
Steve Naroff86272ea2007-05-29 02:14:17 +0000314 Expr *condExpr = (Expr *)Cond;
315 assert(condExpr && "ParseDoStmt(): missing expression");
316
Steve Naroff31090012007-07-16 21:54:35 +0000317 DefaultFunctionArrayConversion(condExpr);
318 QualType condType = condExpr->getType();
Steve Naroff86272ea2007-05-29 02:14:17 +0000319
320 if (!condType->isScalarType()) // C99 6.8.5p2
321 return Diag(DoLoc, diag::err_typecheck_statement_requires_scalar,
322 condType.getAsString(), condExpr->getSourceRange());
323
324 return new DoStmt((Stmt*)Body, condExpr);
Chris Lattneraf8d5812006-11-10 05:07:45 +0000325}
326
327Action::StmtResult
328Sema::ParseForStmt(SourceLocation ForLoc, SourceLocation LParenLoc,
329 StmtTy *First, ExprTy *Second, ExprTy *Third,
330 SourceLocation RParenLoc, StmtTy *Body) {
Steve Naroff86272ea2007-05-29 02:14:17 +0000331 if (First) {
332 // C99 6.8.5p3: FIXME. Need to hack Parser::ParseForStatement() and
333 // declaration support to create a DeclStmt node. Once this is done,
334 // we can test for DeclStmt vs. Expr (already a sub-class of Stmt).
335 }
336 if (Second) {
337 Expr *testExpr = (Expr *)Second;
Steve Naroff31090012007-07-16 21:54:35 +0000338 DefaultFunctionArrayConversion(testExpr);
339 QualType testType = testExpr->getType();
Steve Naroff86272ea2007-05-29 02:14:17 +0000340
341 if (!testType->isScalarType()) // C99 6.8.5p2
342 return Diag(ForLoc, diag::err_typecheck_statement_requires_scalar,
343 testType.getAsString(), testExpr->getSourceRange());
344 }
Chris Lattneraf8d5812006-11-10 05:07:45 +0000345 return new ForStmt((Stmt*)First, (Expr*)Second, (Expr*)Third, (Stmt*)Body);
346}
347
348
349Action::StmtResult
350Sema::ParseGotoStmt(SourceLocation GotoLoc, SourceLocation LabelLoc,
351 IdentifierInfo *LabelII) {
Chris Lattnere2473062007-05-28 06:28:18 +0000352 // Look up the record for this label identifier.
353 LabelStmt *&LabelDecl = LabelMap[LabelII];
354
355 // If we haven't seen this label yet, create a forward reference.
356 if (LabelDecl == 0)
357 LabelDecl = new LabelStmt(LabelLoc, LabelII, 0);
358
359 return new GotoStmt(LabelDecl);
Chris Lattneraf8d5812006-11-10 05:07:45 +0000360}
Chris Lattner1c310502007-05-31 06:00:00 +0000361
Chris Lattneraf8d5812006-11-10 05:07:45 +0000362Action::StmtResult
363Sema::ParseIndirectGotoStmt(SourceLocation GotoLoc,SourceLocation StarLoc,
364 ExprTy *DestExp) {
Chris Lattner1c310502007-05-31 06:00:00 +0000365 // FIXME: Verify that the operand is convertible to void*.
366
Chris Lattneraf8d5812006-11-10 05:07:45 +0000367 return new IndirectGotoStmt((Expr*)DestExp);
368}
369
370Action::StmtResult
Chris Lattnereaafe1222006-11-10 05:17:58 +0000371Sema::ParseContinueStmt(SourceLocation ContinueLoc, Scope *CurScope) {
372 Scope *S = CurScope->getContinueParent();
373 if (!S) {
374 // C99 6.8.6.2p1: A break shall appear only in or as a loop body.
375 Diag(ContinueLoc, diag::err_continue_not_in_loop);
376 return true;
377 }
378
Chris Lattneraf8d5812006-11-10 05:07:45 +0000379 return new ContinueStmt();
380}
381
382Action::StmtResult
Chris Lattnereaafe1222006-11-10 05:17:58 +0000383Sema::ParseBreakStmt(SourceLocation BreakLoc, Scope *CurScope) {
384 Scope *S = CurScope->getBreakParent();
385 if (!S) {
386 // C99 6.8.6.3p1: A break shall appear only in or as a switch/loop body.
387 Diag(BreakLoc, diag::err_break_not_in_loop_or_switch);
388 return true;
389 }
390
Chris Lattneraf8d5812006-11-10 05:07:45 +0000391 return new BreakStmt();
392}
393
394
395Action::StmtResult
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000396Sema::ParseReturnStmt(SourceLocation ReturnLoc, ExprTy *rex) {
397 Expr *RetValExp = static_cast<Expr *>(rex);
Steve Naroff9358c712007-05-27 23:58:33 +0000398 QualType lhsType = CurFunctionDecl->getResultType();
399
Steve Naroff9358c712007-05-27 23:58:33 +0000400 if (lhsType->isVoidType()) {
Steve Naroff6f49f5d2007-05-29 14:23:36 +0000401 if (RetValExp) // C99 6.8.6.4p1 (ext_ since GCC warns)
402 Diag(ReturnLoc, diag::ext_return_has_expr,
403 CurFunctionDecl->getIdentifier()->getName(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000404 RetValExp->getSourceRange());
405 return new ReturnStmt(RetValExp);
Steve Naroff6f49f5d2007-05-29 14:23:36 +0000406 } else {
407 if (!RetValExp) {
408 const char *funcName = CurFunctionDecl->getIdentifier()->getName();
409 if (getLangOptions().C99) // C99 6.8.6.4p1 (ext_ since GCC warns)
410 Diag(ReturnLoc, diag::ext_return_missing_expr, funcName);
411 else // C90 6.6.6.4p4
412 Diag(ReturnLoc, diag::warn_return_missing_expr, funcName);
413 return new ReturnStmt((Expr*)0);
414 }
Steve Naroff9358c712007-05-27 23:58:33 +0000415 }
Steve Naroff6f49f5d2007-05-29 14:23:36 +0000416 // we have a non-void function with an expression, continue checking
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000417 QualType rhsType = RetValExp->getType();
Steve Naroff9358c712007-05-27 23:58:33 +0000418
Steve Naroff6f49f5d2007-05-29 14:23:36 +0000419 // C99 6.8.6.4p3(136): The return statement is not an assignment. The
420 // overlap restriction of subclause 6.5.16.1 does not apply to the case of
421 // function return.
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000422 AssignmentCheckResult result = CheckSingleAssignmentConstraints(lhsType,
423 RetValExp);
Ted Kremenekc48affb2007-08-14 18:14:14 +0000424
Steve Naroff9358c712007-05-27 23:58:33 +0000425 // decode the result (notice that extensions still return a type).
426 switch (result) {
427 case Compatible:
428 break;
429 case Incompatible:
430 Diag(ReturnLoc, diag::err_typecheck_return_incompatible,
431 lhsType.getAsString(), rhsType.getAsString(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000432 RetValExp->getSourceRange());
Steve Naroff9358c712007-05-27 23:58:33 +0000433 break;
434 case PointerFromInt:
435 // check for null pointer constant (C99 6.3.2.3p3)
Chris Lattner0e9d6222007-07-15 23:26:56 +0000436 if (!RetValExp->isNullPointerConstant(Context)) {
Steve Naroff56faab22007-05-30 04:20:12 +0000437 Diag(ReturnLoc, diag::ext_typecheck_return_pointer_int,
438 lhsType.getAsString(), rhsType.getAsString(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000439 RetValExp->getSourceRange());
Steve Naroff9992bba2007-05-30 16:27:15 +0000440 }
Steve Naroff9358c712007-05-27 23:58:33 +0000441 break;
442 case IntFromPointer:
Steve Naroff56faab22007-05-30 04:20:12 +0000443 Diag(ReturnLoc, diag::ext_typecheck_return_pointer_int,
444 lhsType.getAsString(), rhsType.getAsString(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000445 RetValExp->getSourceRange());
Steve Naroff9358c712007-05-27 23:58:33 +0000446 break;
447 case IncompatiblePointer:
Steve Naroff56faab22007-05-30 04:20:12 +0000448 Diag(ReturnLoc, diag::ext_typecheck_return_incompatible_pointer,
449 lhsType.getAsString(), rhsType.getAsString(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000450 RetValExp->getSourceRange());
Steve Naroff9358c712007-05-27 23:58:33 +0000451 break;
452 case CompatiblePointerDiscardsQualifiers:
Steve Naroff56faab22007-05-30 04:20:12 +0000453 Diag(ReturnLoc, diag::ext_typecheck_return_discards_qualifiers,
454 lhsType.getAsString(), rhsType.getAsString(),
Steve Naroffb8ea4fb2007-07-13 23:32:42 +0000455 RetValExp->getSourceRange());
Steve Naroff9358c712007-05-27 23:58:33 +0000456 break;
457 }
Ted Kremenekcff94fa2007-08-17 16:46:58 +0000458
459 if (RetValExp) CheckReturnStackAddr(RetValExp, lhsType, ReturnLoc);
460
Chris Lattneraf8d5812006-11-10 05:07:45 +0000461 return new ReturnStmt((Expr*)RetValExp);
462}
463