<div dir="ltr">Yeah, that's a fairly crappy warning - I'm not sure we could've gotten any sufficiently accurate bug-finding warning out of that code, then...</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jan 16, 2014 at 11:43 PM, Evgeniy Stepanov <span dir="ltr"><<a href="mailto:eugeni.stepanov@gmail.com" target="_blank">eugeni.stepanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">asan_activation.cc(56) : warning C4800: 'int' : forcing value to bool<br>
'true' or 'false' (performance warning)<br>
</div>Sounds like they don't like implicit casting of int to bool...<br>
<div class="im HOEnZb"><br>
On Fri, Jan 17, 2014 at 12:55 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
</div><div class="HOEnZb"><div class="h5">> What was the warning? Could Clang have reasonable caught/warned about this?<br>
><br>
><br>
> On Thu, Jan 16, 2014 at 6:21 AM, Evgeniy Stepanov<br>
> <<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>> wrote:<br>
>><br>
>> Author: eugenis<br>
>> Date: Thu Jan 16 08:21:17 2014<br>
>> New Revision: 199390<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=199390&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=199390&view=rev</a><br>
>> Log:<br>
>> [asan] Fix a build warning on windows.<br>
>><br>
>> Modified:<br>
>>     compiler-rt/trunk/lib/asan/asan_activation.cc<br>
>><br>
>> Modified: compiler-rt/trunk/lib/asan/asan_activation.cc<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/asan_activation.cc?rev=199390&r1=199389&r2=199390&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/asan_activation.cc?rev=199390&r1=199389&r2=199390&view=diff</a><br>

>><br>
>> ==============================================================================<br>
>> --- compiler-rt/trunk/lib/asan/asan_activation.cc (original)<br>
>> +++ compiler-rt/trunk/lib/asan/asan_activation.cc Thu Jan 16 08:21:17 2014<br>
>> @@ -22,8 +22,8 @@ namespace __asan {<br>
>>  static struct AsanDeactivatedFlags {<br>
>>    int quarantine_size;<br>
>>    int max_redzone;<br>
>> -  int poison_heap;<br>
>>    int malloc_context_size;<br>
>> +  bool poison_heap;<br>
>>  } asan_deactivated_flags;<br>
>><br>
>>  static bool asan_is_deactivated;<br>
>><br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>