diff options
author | Eli Friedman <eli.friedman@gmail.com> | 2009-12-03 20:31:57 +0000 |
---|---|---|
committer | Eli Friedman <eli.friedman@gmail.com> | 2009-12-03 20:31:57 +0000 |
commit | 1d6fb1669cebcff72103d7cb6f87ae5dffee8d37 (patch) | |
tree | 578303f29e70f6425fc54e353619b39cf4812be5 /clang/lib/AST/Expr.cpp | |
parent | a3536e23c8b13be76dea4ef0e2d6805b37261740 (diff) | |
download | bcm5719-llvm-1d6fb1669cebcff72103d7cb6f87ae5dffee8d37.tar.gz bcm5719-llvm-1d6fb1669cebcff72103d7cb6f87ae5dffee8d37.zip |
Add recursion guards to ice-checking and evaluation for declrefs, so we
don't infinitely recurse for cases we can't evaluate.
llvm-svn: 90480
Diffstat (limited to 'clang/lib/AST/Expr.cpp')
-rw-r--r-- | clang/lib/AST/Expr.cpp | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp index 28f1bd92f3e..2aca0604444 100644 --- a/clang/lib/AST/Expr.cpp +++ b/clang/lib/AST/Expr.cpp @@ -1596,13 +1596,18 @@ static ICEDiag CheckICE(const Expr* E, ASTContext &Ctx) { // constant expression (5.19). In that case, the member can appear // in integral constant expressions. if (Def->isOutOfLine()) { - Dcl->setInitKnownICE(Ctx, false); + Dcl->setInitKnownICE(false); return ICEDiag(2, cast<DeclRefExpr>(E)->getLocation()); } - + + if (Dcl->isCheckingICE()) { + return ICEDiag(2, cast<DeclRefExpr>(E)->getLocation()); + } + + Dcl->setCheckingICE(); ICEDiag Result = CheckICE(Init, Ctx); // Cache the result of the ICE test. - Dcl->setInitKnownICE(Ctx, Result.Val == 0); + Dcl->setInitKnownICE(Result.Val == 0); return Result; } } |