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.
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
