[llvm] r180851 - get rid of windows warning:

Reid Kleckner rnk at google.com
Wed May 1 09:01:58 PDT 2013


On Wed, May 1, 2013 at 8:12 AM, David Blaikie <dblaikie at gmail.com> wrote:
>
> On May 1, 2013 8:01 AM, "Peng Cheng" <gm4cheng at gmail.com> wrote:
>>
>> Author: chp
>> Date: Wed May  1 10:00:07 2013
>> New Revision: 180851
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=180851&view=rev
>> Log:
>> get rid of windows warning:
>>
>> warning C4800: forcing value to bool 'true' or 'false' (performance
>> warning)
>
> This warning is, in my opinion/experience, particularly unhelpful. Can we
> just disable it instead?

It's already disabled in HandleLLVMOptions.cmake:
http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?view=markup#l186

Is there some reason it's not being picked up for Peng's build?  Is
the header being included in a separate project?



More information about the llvm-commits mailing list