diff options
| author | Mike Stump <mrs@apple.com> | 2009-10-29 20:48:09 +0000 |
|---|---|---|
| committer | Mike Stump <mrs@apple.com> | 2009-10-29 20:48:09 +0000 |
| commit | 980ca220848d27ef55ef4c2d37423461a8ed0da3 (patch) | |
| tree | 3ccd2e56970561c7be62ffcc9c9799d65d5e3e28 /lib/AST/ExprConstant.cpp | |
| parent | f322ed6d39a30f509023cf88588c1e6514226127 (diff) | |
Fix some issues Daniel pointed out.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@85526 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/AST/ExprConstant.cpp')
| -rw-r--r-- | lib/AST/ExprConstant.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/AST/ExprConstant.cpp b/lib/AST/ExprConstant.cpp index b408a3f615..9bf5215634 100644 --- a/lib/AST/ExprConstant.cpp +++ b/lib/AST/ExprConstant.cpp @@ -184,13 +184,14 @@ public: bool VisitCharacterLiteral(CharacterLiteral *E) { return false; } bool VisitSizeOfAlignOfExpr(SizeOfAlignOfExpr *E) { return false; } bool VisitArraySubscriptExpr(ArraySubscriptExpr *E) - { return Visit(E->getLHS()) && Visit(E->getRHS()); } + { return Visit(E->getLHS()) || Visit(E->getRHS()); } bool VisitChooseExpr(ChooseExpr *E) { return Visit(E->getChosenSubExpr(Info.Ctx)); } bool VisitCastExpr(CastExpr *E) { return Visit(E->getSubExpr()); } bool VisitBinAssign(BinaryOperator *E) { return true; } bool VisitCompoundAssign(BinaryOperator *E) { return true; } - bool VisitBinaryOperator(BinaryOperator *E) { return false; } + bool VisitBinaryOperator(BinaryOperator *E) + { return Visit(E->getLHS()) || Visit(E->getRHS()); } bool VisitUnaryPreInc(UnaryOperator *E) { return true; } bool VisitUnaryPostInc(UnaryOperator *E) { return true; } bool VisitUnaryPreDec(UnaryOperator *E) { return true; } @@ -198,7 +199,7 @@ public: bool VisitUnaryDeref(UnaryOperator *E) { if (E->getType().isVolatileQualified()) return true; - return false; + return Visit(E->getSubExpr()); } bool VisitUnaryOperator(UnaryOperator *E) { return Visit(E->getSubExpr()); } }; |