[cfe-commits] r59061 - /cfe/trunk/lib/CodeGen/CGExprScalar.cpp

Chris Lattner sabre at nondot.org
Tue Nov 11 22:35:09 PST 2008


On Nov 11, 2008, at 10:07 PM, Eli Friedman wrote:

> On Tue, Nov 11, 2008 at 8:51 PM, Chris Lattner <sabre at nondot.org>  
> wrote:
>>
>> On Nov 11, 2008, at 1:03 PM, Eli Friedman wrote:
>>
>>> On Tue, Nov 11, 2008 at 10:56 AM, Chris Lattner <sabre at nondot.org>  
>>> wrote:
>>>>
>>>> +    if (CondCI->getZExtValue() == 0)
>>>
>>> This will break once we add support for 128-bit integers.
>>
>> No it won't, because this is guaranteed to be an i1.  However, I'll  
>> change
>> it anyway to clean it up.
>
> No, CondCl is an arbitrary integer.  Note the comment above the
> declaration of CondVal: "Evaluate the conditional, then convert it to
> bool.  We do this explicitly because we need the unconverted value if
> this is a GNU ?: expression with missing middle value."

Ok, well it was safe in the 'if' case, perhaps not in the ?: case.   
All are fixed now, rejoice! :)

-Chris



More information about the cfe-commits mailing list