On Wed, Jun 26, 2013 at 12:25 PM, Aaron Ballman <[email protected]>wrote:
> On Wed, Jun 26, 2013 at 12:22 PM, David Blaikie <[email protected]> > wrote: > > > > On Jun 26, 2013 5:58 AM, "Aaron Ballman" <[email protected]> wrote: > >> > >> Author: aaronballman > >> Date: Wed Jun 26 07:54:49 2013 > >> New Revision: 184941 > >> > >> URL: http://llvm.org/viewvc/llvm-project?rev=184941&view=rev > >> Log: > >> Quieting an MSVC warning about converting negative integer constants to > >> unsigned types. > > > > Looks like a broken warning, unless I'm missing something in the below > code > > (quite possible). Should we just turn it off? > > The warning is kind of broken IMO (in some circumstances), but it > could still catch the occasional bug and doesn't affect us too often > in practice. > 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.
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
