Patch to force SuitableAlign's alignment when loading on object with larger alignment

[David Majnemer]

On Fri, Aug 1, 2014 at 12:43 PM, Reid Kleckner <rnk at google.com> wrote:

> On Fri, Aug 1, 2014 at 12:14 PM, jahanian <fjahanian at apple.com> wrote:
>
>>
>> On Aug 1, 2014, at 11:44 AM, Reid Kleckner <rnk at google.com> wrote:
>>
>> What is HasAlignAttribute, and how is it different from
>> AlignmentIsRequired? I have a feeling that they are different names for the
>> same thing. If you want to rename it, that's fine with me.
>>
>>
>> I first thought I could use AlignmentIsRequired. But then I noticed that
>> AlignmentIsRequired  is set only if current typedef has explicit
>> alignment. (see Decl::getMaxAlignment). We want to make sure that a
>> typedef of an aligned typedef is
>> still considered explicitly aligned (and not to modify its alignment). My
>> patch has a test for this case.
>>
>
> That sounds like a bug.
>

This bug should be fixed in r214698.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140803/995c1cf7/attachment.html>