[llvm-commits] [llvm-gcc-4.2] r72804 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

[Bill Wendling]

On Jun 3, 2009, at 10:18 PM, Duncan Sands wrote:

> Hi Bill,
>
>> Set the alignment of STRING_CSTs to be that which GCC expects it to  
>> be. We were
>> setting it to a default that made the alignment much larger than it  
>> needed to
>> be and wasting space.
>
> I would have expected LLVM to give strings an alignment of 1.  Is that
> not so?
>
No. At this point, before my patch, the alignment wasn't being set.  
During code generation, it defaults (through a series of calls) to a  
"preferred alignment" of 16 bytes.

-bw