diff options
author | Chris Lattner <sabre@nondot.org> | 2009-03-04 04:23:07 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2009-03-04 04:23:07 +0000 |
commit | 24e1e707b4c362f18e371e2bbf054a8345b57bfa (patch) | |
tree | 5489d474d54bca3e549a637ddb1230a0097cf9f2 /lib/AST/StmtSerialization.cpp | |
parent | 074dda6d81315d3e1414ce018325c166bcb02028 (diff) |
Change Parser::ParseCaseStatement to use an iterative approach to parsing
multiple sequential case statements instead of doing it with recursion. This
fixes a problem where we run out of stack space parsing 100K directly nested
cases.
There are a couple other problems that prevent this from being useful in
practice (right now the example only parses correctly with -disable-free and
doesn't work with -emit-llvm), but this is a start.
I'm not including a testcase because it is large and uninteresting for
regtesting.
Sebastian, I would appreciate it if you could scrutinize the smart pointer
gymnastics I do.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@66011 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/AST/StmtSerialization.cpp')
-rw-r--r-- | lib/AST/StmtSerialization.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/AST/StmtSerialization.cpp b/lib/AST/StmtSerialization.cpp index 009a2dd17a..4a90edd7a1 100644 --- a/lib/AST/StmtSerialization.cpp +++ b/lib/AST/StmtSerialization.cpp @@ -414,7 +414,7 @@ void CaseStmt::EmitImpl(Serializer& S) const { CaseStmt* CaseStmt::CreateImpl(Deserializer& D, ASTContext& C) { SourceLocation CaseLoc = SourceLocation::ReadVal(D); CaseStmt* stmt = new (C, llvm::alignof<CaseStmt>()) - CaseStmt(NULL,NULL,NULL,CaseLoc); + CaseStmt(NULL,NULL,CaseLoc); D.ReadPtr(stmt->NextSwitchCase); D.BatchReadOwnedPtrs((unsigned) END_EXPR, &stmt->SubExprs[0], C); return stmt; |