<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 29, 2014 at 7:35 PM, Arthur O'Dwyer <span dir="ltr"><<a href="mailto:arthur.j.odwyer@gmail.com" target="_blank">arthur.j.odwyer@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">FWIW, I don't understand what it means to "assign a value to itself".<br></blockquote><div><br></div><div>
The actual message talks about "assigning value of variable to itself" where the "itself" refers to the variable, not the value. I'm not really worried about this, but I wouldn't object to a change either.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(And if you want to make it something like "assignment does not change<br>
the value of variable %0"</blockquote><div><br></div><div>This diagnostic text also looks fine to me.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
, notice that a^=a actually does change the<br>
value.)  Perhaps "lvalue %0 appears on both sides of (compound)<br>
assignment" would be more accurate.<br>
<br>
I assume this diagnostic is automatically suppressed in the case that<br>
we're really calling an overloaded operator|= or something, right?<br>
<div><div class="h5"><br>
<br>
On Thu, May 29, 2014 at 4:25 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> Maybe tweak the diagnostic (for both this case and the normal self-assignment case) to:<br>
><br>
>   explicitly assigning value of variable of type %0 to itself<br>
><br>
> (Adding the 'value of' makes this message work better for the compound-assignment case.)<br>
><br>
> Other than that, LGTM<br>
><br>
> <a href="http://reviews.llvm.org/D3753" target="_blank">http://reviews.llvm.org/D3753</a><br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>