[PATCH] D22910: Add support for CXXOperatorCallExpr in Expr::HasSideEffects
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 22 05:44:32 PDT 2016
aaron.ballman added a comment.
In https://reviews.llvm.org/D22910#549611, @sylvestre.ledru wrote:
> What about landing this version now (with test) and do the other operators in the a second commit? thanks
I don't see a whole lot of value in that, but I may be missing information -- is this patch gating some other work? Supporting increment and decrement should be pretty straight-forward and is logically related to the current changeset. I'd prefer to fix it all at once since it shouldn't be overly onerous (and it protects us from accidentally only implementing part of the functionality), but if there's a reason to split it out, then I don't see harm in that either.
https://reviews.llvm.org/D22910
More information about the cfe-commits
mailing list