[cfe-commits] NEON vector alignment

[Sandeep Patel]

On Mon, Apr 4, 2011 at 4:58 PM, Bob Wilson <bob.wilson at apple.com> wrote:
>
> On Mar 31, 2011, at 2:06 PM, Sandeep Patel wrote:
>
>> Clang aligns NEON vectors differently than llvm-gcc does.
>>
>> llvm-gcc actually calls into ARMTargetMachine to get the layout, but
>> Clang provides it's own definitions.
>>
>> llvm-gcc's definition appears to be correct, so this changes Clang to that.
>
> Thanks, Sandeep.  While reviewing your patch, I noticed that further changes were needed for the APCS ABI, so I went ahead and made those changes as well.  Committed as svn r128825.
>

Now that you made me look at the parts beyond v128 :-), I'm doubting
the AACPS values for i1,i8, and i16.

The layouts in ARMTargetMachine.cpp for ARM don't have these smaller
integer types listed at all. Should they be explicit?

deep