thakis added a comment.

Thanks! Landing…



================
Comment at: test/SemaCXX/warn-logical-not-compare.cpp:224
+  // These already err, don't also warn.
+  !x &= 1; // expected-error{{expression is not assignable}}
+  !x |= 1; // expected-error{{expression is not assignable}}
----------------
hans wrote:
> And this is the reason you had to put a "not" on the second run line, right?
Right.


https://reviews.llvm.org/D26035



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

Reply via email to