David Majnemer <[email protected]> wrote on 11.07.2015 07:05:14:

>>          // FirstFieldOffset is not appropriate for bitfields,
>>          // it won't tell us what the storage offset should be and
>> thus might not
>>          // be properly aligned.
>>          //
>>          // Instead calculate the storage offset using the offset of
>> the field in
>>          // the struct type.
>>
>> This comment seems out of date now that we are not using the LLVM
>> struct type's layout to calculate FirstByteOffset.

Good point.  I've updated the comment in r242021.

Thanks,
Ulrich


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

Reply via email to