<div dir="ltr">On Wed, Jun 26, 2013 at 12:25 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Jun 26, 2013 at 12:22 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>

><br>
> On Jun 26, 2013 5:58 AM, "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>><br>
>> Author: aaronballman<br>
>> Date: Wed Jun 26 07:54:49 2013<br>
>> New Revision: 184941<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=184941&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=184941&view=rev</a><br>
>> Log:<br>
>> Quieting an MSVC warning about converting negative integer constants to<br>
>> unsigned types.<br>
><br>
> Looks like a broken warning, unless I'm missing something in the below code<br>
> (quite possible). Should we just turn it off?<br>
<br>
</div>The warning is kind of broken IMO (in some circumstances), but it<br>
could still catch the occasional bug and doesn't affect us too often<br>
in practice.<br></blockquote><div><br></div><div>I would say turn it off, but this is in a public header which might be included into someone else's project who doesn't use our warning flags.</div></div></div></div>