<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 24, 2015 at 5:47 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: aaronballman<br>
Date: Tue Mar 24 07:47:51 2015<br>
New Revision: 233067<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=233067&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=233067&view=rev</a><br>
Log:<br>
Silencing some MSVC warnings "C4805: '^' : unsafe mix of type 'bool' and type 'unsigned int' in operation"; NFC.<br></blockquote><div><br>FWIW, Andy Kaylor's started a review/discussion on disabling some MSVC warnings and cleaning up others to get a baseline.<br><br>This is one I'm sort of inclined to disable (as mentioned in the review thread & some other recent discussions - the basic bar that's certainly been applied to GCC and I think is a reasonable one is: If the warning doesn't meet the quality bar to be added to Clang, then it probably doesn't meet the quality bar to be enabled for building LLVM projects and we should just disable the warning rather than fixing it even once)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Modified:<br>
    llvm/trunk/lib/Support/APFloat.cpp<br>
<br>
Modified: llvm/trunk/lib/Support/APFloat.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/APFloat.cpp?rev=233067&r1=233066&r2=233067&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/APFloat.cpp?rev=233067&r1=233066&r2=233067&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Support/APFloat.cpp (original)<br>
+++ llvm/trunk/lib/Support/APFloat.cpp Tue Mar 24 07:47:51 2015<br>
@@ -1430,7 +1430,7 @@ APFloat::addOrSubtractSignificand(const<br>
<br>
   /* Determine if the operation on the absolute values is effectively<br>
      an addition or subtraction.  */<br>
-  subtract ^= sign ^ rhs.sign;<br>
+  subtract ^= static_cast<bool>(sign ^ rhs.sign);<br>
<br>
   /* Are we bigger exponent-wise than the RHS?  */<br>
   bits = exponent - rhs.exponent;<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>
</blockquote></div><br></div></div>