aaron.ballman added a comment.

The other thing this patch is missing are tests, btw.


================
Comment at: lib/AST/Expr.cpp:2869
@@ +2868,3 @@
+    // assignment operator is intended to have a side-effect and other 
overloaded
+    // operators are not. Otherwise fall through the logic of call expression.
+    OverloadedOperatorKind Op = cast<CXXOperatorCallExpr>(this)->getOperator();
----------------
I think that `operator++()` and `operator--()` should be handled as well as the 
assignment operators. @rsmith, thoughts?


https://reviews.llvm.org/D22910



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to