<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 10, 2014 at 5:27 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@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 dir="ltr"><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Fri, Oct 10, 2014 at 4:39 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden">================<br>
Comment at: lib/AST/ExprConstant.cpp:8047-8048<br>
@@ +8046,4 @@<br>
+        APFloat Denom = C * C + D * D;<br>
+        ResR = scalbn((A * C + B * D) / Denom, -DenomLogB);<br>
+        ResI = scalbn((B * C - A * D) / Denom, -DenomLogB);<br>
+        if (ResR.isNaN() && ResI.isNaN()) {<br>
----------------<br>
It looks like `B` is not initialized if `LHSReal` is `true` here.</div></blockquote></div><br></div></div>Right you are. I just need to conjure a proper zero here.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Anything else you see that needs fixing?</div></div>
</blockquote></div><br></div><div class="gmail_extra">No, the rest LGTM.</div></div>