On Aug 3, 2014, at 10:20 PM, David Majnemer <[email protected]> wrote:
> On Fri, Aug 1, 2014 at 12:43 PM, Reid Kleckner <[email protected]> wrote: > On Fri, Aug 1, 2014 at 12:14 PM, jahanian <[email protected]> wrote: > > On Aug 1, 2014, at 11:44 AM, Reid Kleckner <[email protected]> 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. Thanks. - Fariborz
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
