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

[Reid Kleckner]

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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140801/a4f44e63/attachment.html>