[llvm-commits]  When an LLVM type has struct padding that overlaps with important fields of
baldrick at free.fr
Fri Jun 1 15:03:12 CDT 2007
> Okay, do you have a testcase that demonstrates this?
I will synthesize one for you.
> Also, the code for handling "large arrays" isn't needed. This code
> is only called for fixed size structures that are small.
Hey, you are the one who used getInt64 to get the array length! :)
Another point to consider is that this call could barf:
unsigned EltSizeBits = TREE_INT_CST_LOW(TYPE_SIZE(TREE_TYPE(type)));
This can happen if the array is a zero length array with an element
type of variable size (meaning the array has length zero). I reckon the
best thing to do is to bail out early if the type has zero size.
I've attached a new patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1866 bytes
Desc: not available
Url : http://lists.cs.uiuc.edu/pipermail/llvm-commits/attachments/20070601/2ccd7d5d/attachment.bin
More information about the llvm-commits