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.


cfe-commits mailing list

Reply via email to