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

Eli Friedman eli.friedman at gmail.com
Tue Nov 11 22:07:25 PST 2008


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."

-Eli



More information about the cfe-commits mailing list