On Fri, Oct 10, 2014 at 5:27 PM, Chandler Carruth <[email protected]>
wrote:

>
> On Fri, Oct 10, 2014 at 4:39 PM, Richard Smith <[email protected]>
> wrote:
>
>> ================
>> Comment at: lib/AST/ExprConstant.cpp:8047-8048
>> @@ +8046,4 @@
>> +        APFloat Denom = C * C + D * D;
>> +        ResR = scalbn((A * C + B * D) / Denom, -DenomLogB);
>> +        ResI = scalbn((B * C - A * D) / Denom, -DenomLogB);
>> +        if (ResR.isNaN() && ResI.isNaN()) {
>> ----------------
>> It looks like `B` is not initialized if `LHSReal` is `true` here.
>>
>
> Right you are. I just need to conjure a proper zero here.
>
> Anything else you see that needs fixing?
>

No, the rest LGTM.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to