commit 78dd725cded98e0a736fb8ea5db6c4e7d1b819e7
author Richard Trieu <rtrieu@google.com> Wed Sep 24 03:53:56 2014 +0000
committer Richard Trieu <rtrieu@google.com> Wed Sep 24 03:53:56 2014 +0000
tree ea09b466b7d96ffe31b5c8df377be2899bef4ea7
parent b10c92001ccaaeda5154d5c20c23fdaf18f8d9c3

Fix an edge case with BinaryOperator's in -Wuninitialized.  Add testcases for
the other visitors as well.

llvm-svn: 218366

clang/lib/Sema/SemaDecl.cpp

diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index b3098ab..71017f4 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -8359,9 +8359,11 @@
       }
 
       if (BinaryOperator *BO = dyn_cast<BinaryOperator>(E)) {
-        if (BO->getOpcode() == BO_Comma)
+        if (BO->getOpcode() == BO_Comma) {
+          Visit(BO->getLHS());
           HandleValue(BO->getRHS());
-        return;
+          return;
+        }
       }
 
       if (isa<MemberExpr>(E)) {