[llvm-commits] [llvm-gcc-4.2] r46450 - in /llvm-gcc-4.2/trunk/gcc: llvm-abi.h llvm-internal.h llvm-types.cpp

Devang Patel dpatel at apple.com
Mon Jan 28 09:21:05 PST 2008


Hi Duncan,

On Jan 28, 2008, at 2:14 AM, Duncan Sands wrote:

>> +        // Skip 'int:0', which just affects layout.
>> +        unsigned FieldSizeInBits =  
>> TREE_INT_CST_LOW(DECL_SIZE(Field));
>> +        if (FieldSizeInBits == 0)
>> +          continue;
>
> This is probably safe (not completely clear), but why do you need to  
> do it at all?

   getLLVMFieldFor() does not handle zero-sized bit-fields.

-
Devang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20080128/8a5c8099/attachment.html>


More information about the llvm-commits mailing list