[cfe-commits] r70384 - in /cfe/trunk: lib/AST/Expr.cpp test/SemaCXX/unused.cpp

[Eli Friedman]

Author: efriedma
Date: Wed Apr 29 11:35:53 2009
New Revision: 70384

URL: http://llvm.org/viewvc/llvm-project?rev=70384&view=rev
Log:
PR4103: Silence bogus unused expression warning.


Added:
    cfe/trunk/test/SemaCXX/unused.cpp
Modified:
    cfe/trunk/lib/AST/Expr.cpp

Modified: cfe/trunk/lib/AST/Expr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/Expr.cpp?rev=70384&r1=70383&r2=70384&view=diff

==============================================================================
--- cfe/trunk/lib/AST/Expr.cpp (original)
+++ cfe/trunk/lib/AST/Expr.cpp Wed Apr 29 11:35:53 2009
@@ -500,7 +500,8 @@
     return true;
 
   case CallExprClass:
-  case CXXOperatorCallExprClass: {
+  case CXXOperatorCallExprClass:
+  case CXXMemberCallExprClass: {
     // If this is a direct call, get the callee.
     const CallExpr *CE = cast<CallExpr>(this);
     const Expr *CalleeExpr = CE->getCallee()->IgnoreParenCasts();

Added: cfe/trunk/test/SemaCXX/unused.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/unused.cpp?rev=70384&view=auto

==============================================================================
--- cfe/trunk/test/SemaCXX/unused.cpp (added)
+++ cfe/trunk/test/SemaCXX/unused.cpp Wed Apr 29 11:35:53 2009
@@ -0,0 +1,15 @@
+// RUN: clang-cc -fsyntax-only -verify %s
+// PR4103 : Make sure we don't get a bogus unused expression warning
+class APInt {
+  char foo;
+};
+class APSInt : public APInt {
+  char bar;
+public:
+  APSInt &operator=(const APSInt &RHS);
+};
+
+APSInt& APSInt::operator=(const APSInt &RHS) {
+  APInt::operator=(RHS);
+  return *this;
+}