From f961e5921f3027280b733362ffb2e9ac3497752c Mon Sep 17 00:00:00 2001 From: Douglas Gregor Date: Fri, 17 Apr 2009 16:34:57 +0000 Subject: PCH support for return statements. Optimize PCH encoding for switch-case statements slightly, by making the switch-case numbering local to a particular statement. llvm-svn: 69355 --- clang/lib/Frontend/PCHReader.cpp | 13 +++++++++++++ clang/lib/Frontend/PCHWriter.cpp | 9 +++++++++ 2 files changed, 22 insertions(+) (limited to 'clang/lib') diff --git a/clang/lib/Frontend/PCHReader.cpp b/clang/lib/Frontend/PCHReader.cpp index d2fb8ab3cbd..0dcdf8d3fd8 100644 --- a/clang/lib/Frontend/PCHReader.cpp +++ b/clang/lib/Frontend/PCHReader.cpp @@ -256,6 +256,7 @@ namespace { unsigned VisitForStmt(ForStmt *S); unsigned VisitContinueStmt(ContinueStmt *S); unsigned VisitBreakStmt(BreakStmt *S); + unsigned VisitReturnStmt(ReturnStmt *S); unsigned VisitExpr(Expr *E); unsigned VisitPredefinedExpr(PredefinedExpr *E); unsigned VisitDeclRefExpr(DeclRefExpr *E); @@ -398,6 +399,13 @@ unsigned PCHStmtReader::VisitBreakStmt(BreakStmt *S) { return 0; } +unsigned PCHStmtReader::VisitReturnStmt(ReturnStmt *S) { + VisitStmt(S); + S->setRetValue(cast_or_null(StmtStack.back())); + S->setReturnLoc(SourceLocation::getFromRawEncoding(Record[Idx++])); + return 1; +} + unsigned PCHStmtReader::VisitExpr(Expr *E) { VisitStmt(E); E->setType(Reader.GetType(Record[Idx++])); @@ -2172,6 +2180,10 @@ Stmt *PCHReader::ReadStmt() { S = new (Context) BreakStmt(Empty); break; + case pch::STMT_RETURN: + S = new (Context) ReturnStmt(Empty); + break; + case pch::EXPR_PREDEFINED: // FIXME: untested (until we can serialize function bodies). S = new (Context) PredefinedExpr(Empty); @@ -2311,6 +2323,7 @@ Stmt *PCHReader::ReadStmt() { StmtStack.push_back(S); } assert(StmtStack.size() == 1 && "Extra expressions on stack!"); + SwitchCaseStmts.clear(); return StmtStack.back(); } diff --git a/clang/lib/Frontend/PCHWriter.cpp b/clang/lib/Frontend/PCHWriter.cpp index 29c9eb81f44..91490549eab 100644 --- a/clang/lib/Frontend/PCHWriter.cpp +++ b/clang/lib/Frontend/PCHWriter.cpp @@ -458,6 +458,7 @@ namespace { void VisitForStmt(ForStmt *S); void VisitContinueStmt(ContinueStmt *S); void VisitBreakStmt(BreakStmt *S); + void VisitReturnStmt(ReturnStmt *S); void VisitExpr(Expr *E); void VisitPredefinedExpr(PredefinedExpr *E); void VisitDeclRefExpr(DeclRefExpr *E); @@ -592,6 +593,13 @@ void PCHStmtWriter::VisitBreakStmt(BreakStmt *S) { Code = pch::STMT_BREAK; } +void PCHStmtWriter::VisitReturnStmt(ReturnStmt *S) { + VisitStmt(S); + Writer.WriteSubStmt(S->getRetValue()); + Writer.AddSourceLocation(S->getReturnLoc(), Record); + Code = pch::STMT_RETURN; +} + void PCHStmtWriter::VisitExpr(Expr *E) { VisitStmt(E); Writer.AddTypeRef(E->getType(), Record); @@ -1822,6 +1830,7 @@ void PCHWriter::FlushStmts() { } StmtsToEmit.clear(); + SwitchCaseIDs.clear(); } unsigned PCHWriter::RecordSwitchCaseID(SwitchCase *S) { -- cgit v1.2.3