<p dir="ltr"><br>
On Jun 26, 2013 9:25 AM, "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
><br>
> 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">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>
> 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.</p>
<p dir="ltr">Yeah, I don't feel too strongly but our usual approach is to suppress such warnings in other compilers and if there's value, add them to clang (possibly with any tweaks/improvements they need to meet our standards)</p>

<p dir="ltr">><br>
> ~Aaron<br>
</p>