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 /Driver | |
parent | 074dda6d81315d3e1414ce018325c166bcb02028 (diff) | |
download | clang-24e1e707b4c362f18e371e2bbf054a8345b57bfa.tar.gz |
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 'Driver')
-rw-r--r-- | Driver/PrintParserCallbacks.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/Driver/PrintParserCallbacks.cpp b/Driver/PrintParserCallbacks.cpp index b14f2660b8..0a6d09a02e 100644 --- a/Driver/PrintParserCallbacks.cpp +++ b/Driver/PrintParserCallbacks.cpp @@ -272,8 +272,7 @@ namespace { ExprArg LHSVal, SourceLocation DotDotDotLoc, ExprArg RHSVal, - SourceLocation ColonLoc, - StmtArg SubStmt) { + SourceLocation ColonLoc) { llvm::cout << __FUNCTION__ << "\n"; return StmtEmpty(); } |