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
+ // 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?
cfe-commits mailing list