[llvm-commits] [126898] Use one i32 instead of four i8 while emitting

Devang Patel dpatel at apple.com
Fri May 4 16:11:03 PDT 2007


On May 4, 2007, at 3:12 PM, Chris Lattner wrote:

>
> On May 4, 2007, at 2:44 PM, dpatel at apple.com wrote:
>
>> -----------
>> Use one i32 instead of four i8 while emitting
>> tail padding.
>>
>> This addresses first part of PR1278.
>
> Cool.  Does this do the right thing if the end of a struct is not 4-
> byte aligned?

It does not. I reverted this patch for now. Thanks for caching this.

-
Devang

> If the struct requires 7 bytes of tail padding, for
> example, it would often be wrong to emit an i32 then [3 x i8],
> because there will be padding before the i32.





More information about the llvm-commits mailing list